After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 745670 - application menu submenus too wide
application menu submenus too wide
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: extensions
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 702714 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2015-03-05 11:15 UTC by Matthias Clasen
Modified: 2015-03-07 02:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot (26.41 KB, image/png)
2015-03-05 11:15 UTC, Matthias Clasen
  Details
app-menu: Remove unused function (1001 bytes, patch)
2015-03-05 13:57 UTC, Florian Müllner
committed Details | Review
apps-menu: Make it a bit narrower (951 bytes, patch)
2015-03-05 13:58 UTC, Florian Müllner
committed Details | Review
Screenshot using 35em width (49.14 KB, image/png)
2015-03-05 15:54 UTC, Florian Müllner
  Details

Description Matthias Clasen 2015-03-05 11:15:21 UTC
Created attachment 298624 [details]
screenshot

In classic mode (with shell 3.15.91, extensions 3.15.90), I find that the application menu submenus are a bit wide.
Comment 1 Florian Müllner 2015-03-05 11:47:48 UTC
I suspect that's because someone thought that
  "OpenJDK 8 Policy Tool 1.8.0.40-19.b12.fc22.x86_64"
and
  "OpenJDK 8 Monitoring & Management Console 1.8.0.40-19.b12.fc22.x86_64"
where acceptable "application" names ...
Comment 2 Matthias Clasen 2015-03-05 13:06:22 UTC
yes thats it. Can we ellipsize a little earlier ?
Comment 3 Florian Müllner 2015-03-05 13:57:05 UTC
Created attachment 298640 [details] [review]
app-menu: Remove unused function

This was an override for a parent function that hasn't existed
since summer 2013 ...
Comment 4 Florian Müllner 2015-03-05 13:58:07 UTC
Created attachment 298641 [details] [review]
apps-menu: Make it a bit narrower

General rule: if it can fit "GNU Image Manipulation Program", it is
wide enough ...

So turns out it was much simpler - we're just setting a very generous fixed width in the code.
Comment 5 Florian Müllner 2015-03-05 15:34:03 UTC
Comment on attachment 298640 [details] [review]
app-menu: Remove unused function

Attachment 298640 [details] pushed as a39aba4 - app-menu: Remove unused function
Comment 6 Florian Müllner 2015-03-05 15:54:40 UTC
Created attachment 298649 [details]
Screenshot using 35em width
Comment 7 Matthias Clasen 2015-03-05 18:12:04 UTC
looks good to me
Comment 8 Florian Müllner 2015-03-06 16:28:23 UTC
Attachment 298641 [details] pushed as 715f445 - apps-menu: Make it a bit narrower
Comment 9 Florian Müllner 2015-03-07 02:51:26 UTC
*** Bug 702714 has been marked as a duplicate of this bug. ***