Correct text zoom issue in the Content Structure popover#15984
Conversation
afercia
left a comment
There was a problem hiding this comment.
@kjellr thanks! Screenshot including some the first heading in the content structure popover, from Firefox with zoom > text only at very high level:
Looks good to me: the text doesn't overlap.
Ideally, the popover width should be able to adjust a bit depending on the available space. Something like a "range" between a min and max width maybe. Not sure if that would impact the calculation of the popover position which is done via JavaScript. If possible, I'd encourage some exploration in that direction 🙂
The build fails, not sure why.
jorgefilipecosta
left a comment
There was a problem hiding this comment.
The build is passing, and the change looks good to me 👍 I guess we can merge this solution but I agree it would be good to explore a dynamic popup width in other PR/issue.
|
Thank you both! |
Fixes #15357.
As noted in the issue, text in the Content Structure popover currently overlaps when using the browser's text zoom feature. This PR switches the fixed percentage width to use the somewhat more flexible
flex-basisinstead. It also adds a tiny bit of spacing in between each item.Before
Viewed at 100%
Viewed at 200%
After
Viewed at 100% (No changes)
Viewed at 200%