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 360946 - Migrate from GnomeAbout to GtkAboutDialog
Migrate from GnomeAbout to GtkAboutDialog
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Shell
2.8.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
evolution[codecleanup]
Depends on: 350253 380534
Blocks:
 
 
Reported: 2006-10-09 17:34 UTC by Matthew Barnes
Modified: 2013-09-10 13:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (6.53 KB, patch)
2006-10-09 18:02 UTC, Matthew Barnes
none Details | Review
Revised patch (4.80 KB, patch)
2006-12-06 04:13 UTC, Matthew Barnes
accepted-commit_now Details | Review

Description Matthew Barnes 2006-10-09 17:34:54 UTC
Evolution is still using the deprecated GnomeAbout widget from libgnomeui, and should migrate to the newer GtkAboutDialog (available since GTK+ 2.6).

I also noticed that the code is trying to load "evolution-1.5.png" for the logo image on the About dialog, which is why Evolution currently displays the generic GNOME footprint.
Comment 1 Matthew Barnes 2006-10-09 18:02:20 UTC
Created attachment 74367 [details] [review]
Proposed patch

This patch does a few things:

- Uses GtkAboutDialog instead of GnomeAbout.
- Adds a link to the Evolution website [1] on the About dialog.
- Uses the correct logo filename (generated by the build tools).
- Polishes some autoconf/automake details related to the logo file.

[1] http://www.gnome.org/projects/evolution/
Comment 2 André Klapper 2006-10-09 21:49:08 UTC
three cheers! :-)
Comment 3 Matthew Barnes 2006-11-19 17:00:45 UTC
This is a proposed GNOME Goal:
http://live.gnome.org/GnomeGoals/AboutDialog
Comment 4 Matthew Barnes 2006-12-04 12:55:36 UTC
This bug requires clarification of Evolution's library requirements.
Please see bug #380534.
Comment 5 Matthew Barnes 2006-12-06 04:13:18 UTC
Created attachment 77781 [details] [review]
Revised patch

Here's a much cleaner patch:

- Uses GtkAboutDialog instead of GnomeAbout.
- Adds a link to the Evolution website [1] on the About dialog.
- Specifies the logo by name ("evolution") instead of loading a pixbuf.
- Evolution windows default to the "evolution" icon instead of "stock_mail".
- No autoconf/automake changes.
- Satisfies GNOME Goal #2.1: Install theme-friendly icons [1]

[1] http://live.gnome.org/GnomeGoals/AppIcon
Comment 6 Matthew Barnes 2006-12-06 04:16:35 UTC
Also, the old Ximian logo (art/evolution.png) is now obsolete.

Maybe we should stop shipping it?
Comment 7 Matthew Barnes 2006-12-06 11:58:16 UTC
Evolution should be shipping the "evolution" icons itself (bug #350253).

I'm making this bug depend on it.
Comment 8 Matthew Barnes 2007-02-08 15:29:42 UTC
Ping: this would add some nice polish to Evo 2.10.
Comment 9 Matthew Barnes 2007-02-12 18:25:44 UTC
Committed to HEAD.