-
Notifications
You must be signed in to change notification settings - Fork 1.3k
modernizing the ui #2073
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
modernizing the ui #2073
Conversation
Some improvements I can think of, @To1ne if you could approve of these I can get started on them :
Current performance (Used Lighthouse): ![]() |
@Paulie-Aditya That's not that bad actually.
Yes, sounds like a good one. Although I'm not sure where we have code blocks? On the Downloads page? Keep in mind it's probably harder on the book pages, because those are imported from another project.
So you want to increase size? I'm supportive of that.
Typography in general can use some love. I'm supportive of all these efforts. Although I'm not an expert on the topic. |
@Paulie-Aditya I see that the ![]() |
fixed, thanks!
I'll try and implement on the docs pages, and also on the books pages if I am able to. Other than that there's just one code block in downloads page, which I will implement
will be working on the changes listed above asap! |
@Paulie-Aditya The workflow to deploy to GitHub Pages on your fork is quite awkward. Locally you can keep using branches like normal, and also to create a PR you'll use those branches. Only to deploy things you need to push to git push origin --force HEAD:gh-pages This pushes |
Yeah currently I run the workflow via hugo and node and test out my changes there itself. |
deployment is now working as expected, I'll push some changes today.. |
Awesome! I see it working: https://paulie-aditya.github.io/git-scm.com/ Looking forward to that new PR. |
Changes
Working on Modernizing the visuals of the website.
Context
#2060