-
Notifications
You must be signed in to change notification settings - Fork 15
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
FIX Nicely truncate link title/URL #153
FIX Nicely truncate link title/URL #153
Conversation
9c5e614
to
88f1b7f
Compare
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 ended up using https://css-tricks.com/flexbox-truncated-text/ as a guide for how to do this.
&::before { | ||
|
||
&[class*="font-icon-"]::before { |
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.
Required to override existing styles due to specificity
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.
By "spacing" I assume you mean the vertical alignment of the text, and not any horizontal padding or anything else? |
88f1b7f
to
d3ece30
Compare
Fixed vertical alignment of versioned badge |
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.
Tested locally, works good
Works regardless of screen size - check with split view and in edit view.
Also check with "DRAFT", "MODIFIED", and published (no) versioned labels.
Before:
After:
Issue