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 547952 - Don't display source groups
Don't display source groups
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: general
2.24.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2008-08-15 19:08 UTC by Michael Monreal
Modified: 2013-09-14 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot (9.43 KB, image/png)
2008-08-15 19:11 UTC, Michael Monreal
  Details
Proposed patch (1.11 KB, patch)
2008-08-15 23:58 UTC, Matthew Barnes
committed Details | Review

Description Michael Monreal 2008-08-15 19:08:14 UTC
The address book and the calendar component have various categories:

Address book
- On this computer
- On LDAP servers
- WebDAV
- Google

Calendar
- On this computer
- Contacts
- On the web
- CalDAV
- Weather
- Google

Most users will not have an entry in all of those groups, still empty groups are shown, which looks bad IMHO. Empty groups can not even be used to create new entries inside them so they are essentially useless.

It would be nice if those group headers would only be shown if there's actually something inside.
Comment 1 Michael Monreal 2008-08-15 19:11:11 UTC
Created attachment 116696 [details]
Screenshot
Comment 2 Matthew Barnes 2008-08-15 19:39:52 UTC
I've been wanting to fix this for _ages_.  Do you think this would qualify as a UI change?  Freeze is Monday.  (One could argue it's a _bug_, not a change.)
Comment 3 Michael Monreal 2008-08-15 19:49:11 UTC
I would argue it's a usability bug because it could potentially confuse users :)

I think you would be fine to fix this even after monday, would be great to have this fixed before 2.24.0.
Comment 4 Matthew Barnes 2008-08-15 23:58:23 UTC
Created attachment 116715 [details] [review]
Proposed patch

Figures.  Took all of about ten minutes (mostly to compile and test).

Deleting the last source in a group still leaves behind an empty group until restart.  Probably something signal-related: not listening for a "row-deleted" tree view signal or something like that.  I'm not gonna worry about it for now.
Comment 5 Michael Monreal 2008-08-16 08:05:55 UTC
Just testing this against trunk. Seems to work fine and gives a lot of polish for such a small patch.

Groups show up if needed without a restart (e.g. you add a new google source when you did not have one before) which is the important case.
Comment 6 Srinivasa Ragavan 2008-08-18 03:11:56 UTC
Looks fine to me. I dont think it is a UI Change.
Comment 7 Matthew Barnes 2008-08-18 03:27:26 UTC
Committed to trunk (revision 9362).