[CLOSED] Fix bug #4593 (Text wrap in context menus & HTML main menus)

[core-ai-bot]

Issue by peterflynn
Saturday Oct 19, 2013 at 00:53 GMT
Originally opened as adobe/brackets#5580

---

See my comments in #4593 for all the gory details...

This fix preserves the current layout, where label & shortcut are not strictly separated columns -- notice how the two columns interlock like puzzle pieces in the working set context menu, for example. But the downside is that this solution may not work in other browsers.

So eventually we may want a more cross-browser fix... but in the meantime this will be a real improvement for the Linux build (which has no native menubar yet).

---

peterflynn included the following code: https://github.com/adobe/brackets/pull/5580/commits

[core-ai-bot]

Comment by peterflynn
Saturday Oct 19, 2013 at 00:54 GMT

---

See screenshot in #5411 for an example of how nasty the menus are on Linux right now.

[core-ai-bot]

Comment by JeffryBooher
Tuesday Oct 22, 2013 at 23:45 GMT

---

Soooooo much better! Thanks@peterflynn. Merging!

[core-ai-bot]

Comment by JeffryBooher
Wednesday Oct 23, 2013 at 23:31 GMT

---

FBNC@peterflynn

[core-ai-bot]

Comment by JeffryBooher
Wednesday Oct 23, 2013 at 23:31 GMT

---

Actually,@peterflynn I'm just going to close it since you opened it and fixed it.

[core-ai-bot]

Comment by peterflynn
Wednesday Oct 23, 2013 at 23:43 GMT

---

I assume you meant the bug itself, not the PR? But yeah, I'm fine with closing -- will go ahead & do that now.