Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Speed up the landing page #204

Merged
merged 17 commits into from
Aug 4, 2022
Merged

Speed up the landing page #204

merged 17 commits into from
Aug 4, 2022

Conversation

baylessj
Copy link
Contributor

@baylessj baylessj commented Dec 6, 2020

Moving away from Bootstrap and the old template to a simple CSS file and no JS. Kept the home page mostly the same, but added a couple of new items too.

Lighthouse scores (on localhost)

  • Mobile: 100
  • Desktop: 99

Screenshots:

Desktop
Screenshot 2022-06-06 at 11-42-26 PROS

Mobile
Screenshot 2022-06-06 at 11-42-56 PROS

@baylessj
Copy link
Contributor Author

@WillXuCodes
Copy link
Member

https://www.vexforum.com/t/the-pros-of-pros/59425/14?u=ranchsauce

Another testimonial (though do note it's from theo lol)

@WillXuCodes
Copy link
Member

By the way, I feel like we should notify those who we are using testimonials of in some capacity or another that we are using their quotes on our webpage.

@baylessj
Copy link
Contributor Author

I like your post from Theo best, let's go with that one. I can reach out to him and Oscar on discord to get their permission to use the testimonials.

@baylessj
Copy link
Contributor Author

Both Theo and Oscar are cool with the quotes, I'll update the PR with them in a bit.

@baylessj
Copy link
Contributor Author

20170409_201526
20171117_224502

So here are the two best photo candidates I could find for the PROS photo... They're all from 2017 though and not great candidates hahaha

It seems like we're not getting much from the request for photos in our discord so it might be time to strike that section from the draft

@WillXuCodes
Copy link
Member

Honestly, the first picture would probably be the better of the two candidates. However, another good alternative to this is to put the picture of Purdue that we already have in that spot.

@baylessj
Copy link
Contributor Author

baylessj commented Dec 21, 2020

I tried using the Purdue picture but it seemed a bit empty, was best off as a background image. I pulled the header image from this article about the tournament (https://www.cs.purdue.edu/news/articles/2019/SIGBots-2019.html), thoughts?

Screenshot from 2020-12-20 20-20-08

@WillXuCodes
Copy link
Member

Oh yeah that looks better for sure, good with me 👍

@baylessj
Copy link
Contributor Author

@kunwarsahni01 @HotelCalifornia This PR should be ready to go so long as it's good with you, can you add your reviews whenever you get a chance?

@HotelCalifornia
Copy link
Contributor

Any chance we could re-record the gif in the background with the latest API? if not it's whatever

@HotelCalifornia
Copy link
Contributor

also maybe as a side project, set up auto-deploy forCI build artifacts from PRs to a staging URL so the results can be verified online (:

@baylessj
Copy link
Contributor Author

Any chance we could re-record the gif in the background with the latest API? if not it's whatever

It would be possible but I don't have a Windows machine up and running at the moment, would need someone else to update the GIF with the PROS editor. We could open a separate issue for that here and get around to it eventually maybe?

also maybe as a side project, set up auto-deploy forCI build artifacts from PRs to a staging URL so the results can be verified online (:

I would love to see that happen, not really sure what all we have in the way of a staging domain though. My first thought was to use github pages or something similar, could we do this with the existing azure-pipelines setup though?

@HotelCalifornia
Copy link
Contributor

I would love to see that happen, not really sure what all we have in the way of a staging domain though. My first thought was to use github pages or something similar, could we do this with the existing azure-pipelines setup though?

my initial thought was to set up a branch on this repo for gh-pages to use and then have the pipeline commit the artifacts there under a folder keyed to the PR so you'd be able to get to the build for a PR at https://purduesigbots.github.io/pros-docs/204/ or something. i'm sure there's a way to do this in azure, but failing all else i have some sample github workflows we might be able to adapt

@baylessj
Copy link
Contributor Author

Opened #214 for this, can take a look at that sometime this week

@baylessj
Copy link
Contributor Author

baylessj commented Jan 3, 2021

Kunwar's suggestions from Discord:

  • Fix mobile Chrome disaster -- turns out this is due to the Chrome dev tools phone simulation, works fine on my real iphone
  • Switch to the high res main logo
  • Keep Hero subtitle text white on light theme
  • Fix the left shift of the hero content when navigating forwards/backwards
  • Add a hamburger menu to the mobile version of the site

@baylessj
Copy link
Contributor Author

baylessj commented Jun 6, 2022

I merged master in, updated a couple of things (like mentioning VSCode instead of Atom), and updated the screenshots in the PR description.

For some added context on why I'd really like to merge this in, here's how the current webpage renders (after some time) on my machine:

Screenshot 2022-06-06 at 11-43-07 PROS
Screenshot 2022-06-06 at 11-44-01 PROS

@kunwarsahni01
Copy link
Member

kunwarsahni01 commented Jun 6, 2022

Testing this locally on a mac and looks like the download and getting started buttons link to the same page? Also looks like every time I reload the webpage locally the entire site starts drifting to the left more and more.

@baylessj
Copy link
Contributor Author

baylessj commented Jun 6, 2022

Thanks for the feedback, I also realized that those download/getting started links go to the old, pre-VSC docs. Updates those to match the current stuff

Copy link
Member

@WillXuCodes WillXuCodes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kunwarsahni01 Our front page needs a bit of a touchup, let's get this done.

@baylessj baylessj merged commit 78abe91 into master Aug 4, 2022
@baylessj baylessj deleted the homepage-speedup branch August 4, 2022 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants