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 359581 - GtkCellView clobbers parent's GdkWindow on style-set
GtkCellView clobbers parent's GdkWindow on style-set
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
2.10.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2006-10-04 12:59 UTC by Xan Lopez
Modified: 2006-10-06 18:42 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16

Attachments
Testcase (2.62 KB, text/plain)
2006-10-04 13:06 UTC, Xan Lopez 		Details

Description Xan Lopez 2006-10-04 12:59:41 UTC 
GtkCellView style_set method sets the background of widget->window. Being a NO_WINDOW widget this means its parent's window gets clobbered. Kris suggests to remove the style_set method because it seems to be a leftover from the days when the cellview was a !NO_WINDOW widget and the widget itself is not creating any additional GdkWindow which would need to be notified about the change.
Comment 1 Xan Lopez 2006-10-04 13:06:33 UTC 
Created attachment 74004 [details]
Testcase

Pressing any key you can change the widget inside the window form a ComboBox to a ComboBoxEntry and viceversa. With a ComboBox, the debian pattern in the background is clobbered by the CellView style_set method.
Comment 2 Matthias Clasen 2006-10-06 18:18:59 UTC 
Hmm, I guess thats right. A no-window widget should not set the background of its parent window.
Comment 3 Matthias Clasen 2006-10-06 18:42:20 UTC 
2006-10-06  Matthias Clasen  <mclasen@redhat.com>

        * gtk/gtkcellview.c: Don't set the background of the
        window in a no-window widget.  (#359581, Xan Lopez)