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 484306 - Icon alignment in vertical button lists
Icon alignment in vertical button lists
Status: RESOLVED WONTFIX
Product: evolution
Classification: Applications
Component: general
2.12.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
evolution[cleanup]
Depends on:
Blocks:
 
 
Reported: 2007-10-07 03:32 UTC by Matthew Barnes
Modified: 2009-09-05 01:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot: Message Filters (Before) (26.69 KB, image/png)
2007-10-07 03:33 UTC, Matthew Barnes
  Details
Screenshot: Message Filters (After) (27.61 KB, image/png)
2007-10-07 03:33 UTC, Matthew Barnes
  Details
Patch for Message Filters / Search Folders dialogs (48.73 KB, patch)
2007-10-12 00:39 UTC, Matthew Barnes
none Details | Review

Description Matthew Barnes 2007-10-07 03:32:21 UTC
While the HIG doesn't mention this specifically, I find it more aesthetically pleasing when icons are aligned in a vertical list of buttons, similar to what you'd see on a menu.  I think it's in the spirit of what the HIG *does* say about vertical alignment.  Look at the attached screenshots of Evolution's "Message Filters" dialog, for example.  I think the second screenshot makes the dialog look much more polished.

This kind of thing is of course Srini's domain, so I'll defer to him to decide whether or not we should do this.  If he agrees, I can use this bug as a tracker for fixing up other dialogs in Evolution.

p.s. Is there any reason why we couldn't reinstate the UI category in Bugzilla
     for widget or HIG-related issues like this?  Or does it tend to attract
     too many crazy ideas from users?
Comment 1 Matthew Barnes 2007-10-07 03:33:04 UTC
Created attachment 96793 [details]
Screenshot: Message Filters (Before)
Comment 2 Matthew Barnes 2007-10-07 03:33:34 UTC
Created attachment 96794 [details]
Screenshot: Message Filters (After)
Comment 3 Matthew Barnes 2007-10-12 00:39:26 UTC
Created attachment 97100 [details] [review]
Patch for Message Filters / Search Folders dialogs

It would probably help if I attached the patch...

The patch is just Glade changes.  I opened it in Glade 3 so of course it rewrote a bunch of stuff, but the manual changes are limited to the dialog buttons.
Comment 4 Gilles Dartiguelongue 2007-10-12 07:24:44 UTC
I have a script for you that might help if you know what to change manually :)

anyway, I fill it's appropriate for this dialog because of the number of buttons but then what happens to the other ton of dialogs evolution has and where content of buttons are centered ?
Comment 5 Matthew Barnes 2007-10-12 12:14:50 UTC
(In reply to comment #4)
> I have a script for you that might help if you know what to change manually :)

Please attach it for email me directly.  I'd appreciate it.


> anyway, I fill it's appropriate for this dialog because of the number of
> buttons but then what happens to the other ton of dialogs evolution has and
> where content of buttons are centered ?

The patch is just a sample.  If Srini approves it, I'll use this bug as a tracker for fixing the rest of the dialogs.  Feel free to help out if you want.
Comment 6 Srinivasa Ragavan 2007-10-15 08:22:25 UTC
Matthew, I think it is fine to go this way. I don't have to review this. Just be cautious on the commits though :)
Comment 7 Matthew Barnes 2007-10-15 11:32:15 UTC
Thanks Srini.  I'll fix up some other dialogs and post screenshots here before committing this.  I'd rather commit everything at once to make sure we keep the UI consistent.
Comment 8 Suman Manjunath 2008-04-17 08:12:06 UTC
Matt, do you have a list of other dialogs to be improved? This task could be taken up by NOSIP students to get comfortable with the code. 
Comment 9 Matthew Barnes 2008-04-17 11:56:27 UTC
Yeah, this one kind of got away from me.  I'd say start by rummaging through all the Glade files in Evolution and libedataserverui.  Those are easiest to fix.  Then we can look for any hand-coded dialogs that still need fixing.
Comment 10 Matthew Barnes 2008-04-17 11:57:55 UTC
Oh, and for this bug, patches should be accompanied by before and after screenshots for each dialog.
Comment 11 Matthew Barnes 2008-08-05 03:24:29 UTC
Setting the patch status to Obsolete for now so it doesn't attract attention.  Haven't gotten around to seeing this through yet, and I want to have /all/ the changes ready to go before we commit anything.  The patch is still valid but incomplete, and was meant to just to get the ball rolling.
Comment 12 Matthew Barnes 2009-09-05 01:27:00 UTC
Given the decision to default to icon-free buttons throughout the desktop, I don't think it makes sense to pursue this anymore.  Closing as WONTFIX.