-
Notifications
You must be signed in to change notification settings - Fork 176
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
<502> Remove jump when switching page #610
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.
looks pretty good, its a bit odd that the text visually expands from the top left corner but im not sure how easy that is to fix
Screen.Recording.2024-03-22.at.2.09.37.PM.mov
@rauhul I didn't even notice that. Can you please try now? |
@swift-ci please test |
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.
nice! no jumping and expanding from the center, thanks for fixing this :D
a::before { | ||
display: block; | ||
content: attr(data-text); | ||
font-weight: bold; | ||
height: 0; | ||
overflow: hidden; | ||
visibility: hidden; | ||
} |
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.
Will screen readers pick this duplicate text up?
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.
I did test with VoiceOver and it doesn't pick the duplicate text
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.
@swift-ci please test |
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.
Using a pseudo-element is a clever trick. I am not convinced that bolding the active link is worth the trouble compared to other ways to visually differentiate the active link, but it solves the jumpiness problem for now 👍
2f8473f
to
b21c613
Compare
@swift-ci please test |
@alexandersandberg can you please review so we can merge? |
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.
Apologies for the slow response — looks good!
Motivation:
#502
Modifications:
Added style in:
assets/stylesheets/_screen.scss
Result:
The menu items are not shifting after the page change