RESOLVED FIXED 16378
Implement Icon for Gtk
https://bugs.webkit.org/show_bug.cgi?id=16378
Summary Implement Icon for Gtk
Christian Dywan
Reported 2007-12-10 03:52:37 PST
When looking at the "file" control I noticed that the class Icon that is used to display a file icon is not implemented.
Attachments
Implement Icon for Gtk (2.96 KB, patch)
2007-12-10 03:55 PST, Christian Dywan
alp: review-
Use cairo, with Changelog and explanation (4.44 KB, patch)
2007-12-11 03:46 PST, Christian Dywan
no flags
Updated patch (4.35 KB, patch)
2007-12-17 07:17 PST, Christian Dywan
alp: review+
Christian Dywan
Comment 1 2007-12-10 03:55:33 PST
Created attachment 17818 [details] Implement Icon for Gtk This implementation creates a GdkPixbuf for the icon and probes the icon theme for an appropriate icon according to the icon naming specification or to gnome's icon names respectively.
Alp Toker
Comment 2 2007-12-10 05:37:58 PST
Comment on attachment 17818 [details] Implement Icon for Gtk I didn't check the icon name code yet, but the drawing should almost certainly be done with cairo and gdk_cairo_set_source_pixbuf() rather than gdk_draw_pixbuf() so that the drawing operation will integrate with the rest of the graphics backend. A ChangeLog, particularly to explain lookupIconName() (which was non-obvious to me) would be helpful. Thanks!
Christian Dywan
Comment 3 2007-12-11 03:46:50 PST
Created attachment 17843 [details] Use cairo, with Changelog and explanation
Alp Toker
Comment 4 2007-12-15 09:04:48 PST
What site or test case can this patch be checked against?
Christian Dywan
Comment 5 2007-12-17 07:17:15 PST
Created attachment 17962 [details] Updated patch You can test the code by navigating to any website containing a "file" control. As soon as a file is chosen, an icon is displayed near the filename.
Alp Toker
Comment 6 2007-12-17 08:12:57 PST
Comment on attachment 17962 [details] Updated patch r=me Will add the save/restore, m_icon initialization and style cleanups before landing.
Alp Toker
Comment 7 2007-12-17 08:20:04 PST
Landed in r28809.
Note You need to log in before you can comment on or make changes to this bug.