-
Notifications
You must be signed in to change notification settings - Fork 0
Update webpage for a modern typography-first design #28
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
Conversation
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.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: flaticols <7881854+flaticols@users.noreply.github.com>
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.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| font-variation-settings: 'MONO' var(--mono-axis), 'CASL' var(--casl-axis); | ||
| font-weight: var(--font-weight-regular); | ||
| line-height: 1.6; | ||
| line-height: var(--line-height-relaxed); |
Copilot
AI
Nov 5, 2025
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.
[nitpick] The --line-height-relaxed variable is set to 1.75, but the original value was 1.6. This increases the line height by 9.4%, which may make the text less dense and potentially affect the overall layout. Consider if this change is intentional and aligns with the design goals.
|
|
||
| .change-group li::before { | ||
| content: "•"; | ||
| content: "—"; |
Copilot
AI
Nov 5, 2025
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.
[nitpick] Changed bullet character from '•' to '—' (em dash). This significantly changes the visual appearance of list items. Consider whether an em dash is the most appropriate choice for list bullets, as it's typically used for punctuation rather than list formatting.
| content: "—"; | |
| content: "•"; |
flaticols
left a comment
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.
looks good to me
Redesigning the bump webpage to be minimalistic and typography-first
Design Changes
Before:

After:

The webpage now features:
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.