-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Conversation
Works for me. Would be nice to fix it. |
This CSS change will affect code the hint popup's width too, so we should be careful to test that as well. |
EWF had the same problem with long font names in code hints. There we chose to use an ellipsis, |
I don't really like this idea, because maybe in some language you can't guess what it means when it is truncated. |
The CSS change applies to the wrong part of the HTML menu bar ( |
@martinstarman I'm going to see if I can come up with anything more general. At first glance it seems like the float approach we're currently using should 'just work' without any width tricks needed... http://jsfiddle.net/rGwNJ/ Given that it used to work in the main menus at least, I'm guessing some unrelated CSS update (or our Bootstrap upgrade) broke stuff unexpectedly. |
@peterflynn probably yes, but I played with it for 3 days and I have not found another solution. |
@martinstarman I was able to come up with a more general solution which was landed in PR #5580, so I'm closing this for now. My fix may not work across browsers, so we may have to revisit this down the road -- I put some detailed notes in #4593 for future reference, though. |
Fix for context menu when strings is longer than menu width.
After this patch
Plus remove some unnecessary spaces.