-
Notifications
You must be signed in to change notification settings - Fork 60
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
Conversation
https://www.vexforum.com/t/the-pros-of-pros/59425/14?u=ranchsauce Another testimonial (though do note it's from theo lol) |
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. |
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. |
Both Theo and Oscar are cool with the quotes, I'll update the PR with them in a bit. |
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. |
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? |
Oh yeah that looks better for sure, good with me 👍 |
@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? |
Any chance we could re-record the gif in the background with the latest API? if not it's whatever |
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 (: |
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?
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 |
Opened #214 for this, can take a look at that sometime this week |
Kunwar's suggestions from Discord:
|
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. |
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 |
There was a problem hiding this 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.
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)
Screenshots:
Desktop
Mobile