fix top menu feedback on mobile #157
Conversation
| display: block; | ||
| } | ||
|
|
||
| .menu-item--level-1 > .is-active { |
There was a problem hiding this comment.
The CSS property values for the mobile media query that match the ones on the main interledger.org website are the following (and are identical to those for desktop):
text-decoration-thickness: 2px;
text-underline-offset: 8px;
To avoid repetition, consider moving these shared rules outside the media query.
There was a problem hiding this comment.
Okay, Thanks @Anca2022, have made the changes
There was a problem hiding this comment.
This is on interledger.org/developers (styled at 5px underline-offset):
This is on interledger.org/open-standards (styled at 8px underline-offset) - this is what we want to replicate:
As I mentioned previously, on mobile the values are the same as on desktop:
text-decoration-thickness: 2px;
text-underline-offset: 8px;
Anca2022
left a comment
Anca2022
left a comment
There was a problem hiding this comment.
Looks good
Thank you :D
2 participants
PR Checklist
Summary
Now the UI looks like this
