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 637081 - Port EMap widget to GTK+ 3
Port EMap widget to GTK+ 3
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
3.0.x (obsolete)
Other All
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
evolution[gtk3]
Depends on:
Blocks: 636677
 
 
Reported: 2010-12-12 14:51 UTC by Milan Bouchet-Valat
Modified: 2013-09-13 01:10 UTC
See Also:
GNOME target: 3.0
GNOME version: ---


Attachments
Port EMap widget to GtkScrollable (18.26 KB, patch)
2010-12-12 14:51 UTC, Milan Bouchet-Valat
reviewed Details | Review
Port EMap widget to height-for-width API (2.29 KB, patch)
2010-12-12 14:51 UTC, Milan Bouchet-Valat
none Details | Review
Use GdkVisual instead of GdkColormap in EMap widget (1.35 KB, patch)
2010-12-12 14:51 UTC, Milan Bouchet-Valat
none Details | Review
Use GtkWidget::draw in EMap widget (2.46 KB, patch)
2010-12-12 14:51 UTC, Milan Bouchet-Valat
none Details | Review

Description Milan Bouchet-Valat 2010-12-12 14:51:46 UTC
I ported the EMap widget to GTK+ 3 for the gnome-system-tools. Attached are
the changes I made, adapted to Evolution. These all require GTK3 (they
won't be added to GTK+ 2.x), so I'm mostly dumping them here not to lose them,
but of course you cannot commit them until Evo switches completely.
Comment 1 Milan Bouchet-Valat 2010-12-12 14:51:48 UTC
Created attachment 176288 [details] [review]
Port EMap widget to GtkScrollable

Needed to build with GTK3.
Comment 2 Milan Bouchet-Valat 2010-12-12 14:51:51 UTC
Created attachment 176289 [details] [review]
Port EMap widget to height-for-width API

Needed by GTK3.
Comment 3 Milan Bouchet-Valat 2010-12-12 14:51:54 UTC
Created attachment 176290 [details] [review]
Use GdkVisual instead of GdkColormap in EMap widget

GdkColormap no longer exists in GTK3.
Comment 4 Milan Bouchet-Valat 2010-12-12 14:51:57 UTC
Created attachment 176291 [details] [review]
Use GtkWidget::draw in EMap widget

GtkWidget ::expose-event no longer exists in GTK3.

We don't need to check gtk_widget_is_drawable() because GTK does it
for us.
Comment 5 Milan Bouchet-Valat 2010-12-12 15:01:14 UTC
Some lines look ugly because tabs are replaced with spaces, but the file isn't consistent at all in that regard (if you look closely, you'll notice Company's work is the culprit ;-). Running indent after that could be a good idea.
Comment 6 Milan Crha 2011-01-21 18:34:15 UTC
Thanks for patches. The code changed quite much since you've done this. So it

a) doesn't apply cleanly to gtk3 branch of evolution,
   which is gtk3 only already.
b) the last three patches are obsolete in that branch
c) even when I add changes from the first patch then it doesn't compile.

I'll try to fix it next week.
Comment 7 Milan Crha 2011-01-21 18:40:43 UTC
Created commit 69f3641 in gtk3 branch (maybe 2.91.6+, or slightly later)