readthedocs / addons

JavaScript client to integrate with Read the Docs nicely
https://readthedocs-addons.readthedocs.io/
MIT License
22 stars 6 forks source link

Don't wrap the flyout header. #440

Open ericholscher opened 1 week ago

ericholscher commented 1 week ago

This makes it so the version doesn't overflow as badly:

Before

Screenshot 2024-11-19 at 2 30 24 PM

After

Screenshot 2024-11-19 at 2 30 12 PM
agjohnson commented 1 week ago

Unfortunately, both options have issues. I don't think we should shrink the flex grid down without addressing content widths of the labels

While at least it doesn't appear under the logo, without flex grid wrapping the flyout content becomes scrollable and the long strings now wrap oddly and can look worse with icons:

image image image

We should truncate the text to fix this: https://css-tricks.com/flexbox-truncated-text/