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 684171 - Patch to make use-layout-size property work properly
Patch to make use-layout-size property work properly
Status: RESOLVED FIXED
Product: clutter-gtk
Classification: Platform
Component: GtkClutterEmbed
unspecified
Other Linux
: Normal normal
: ---
Assigned To: clutter-gtk maintainer(s)
clutter-gtk maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-09-17 04:27 UTC by Tristan Van Berkom
Modified: 2012-10-17 22:09 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch to call gtk_widget_queue_resize at the appropriate time (2.55 KB, patch)
2012-09-17 04:27 UTC, Tristan Van Berkom 		committed Details | Review

Description Tristan Van Berkom 2012-09-17 04:27:00 UTC 
Created attachment 224472 [details] [review]
Patch to call gtk_widget_queue_resize at the appropriate time

Currently use-relayout-size only works once, which means if you're content
changes it's size the GtkClutterEmbed will not renegotiate it's size accordingly
(currently I have a GtkClutterEmbed inside a GtkScrolledWindow and viewport
which seems to work the first time, but when I add/remove contents then
the scrolled window does not update the content size).
Comment 1 Emmanuele Bassi (:ebassi) 2012-10-17 14:22:26 UTC 
I'd like to have Boxes test this, given that they introduced the feature in the first place.
Comment 2 Cosimo Cecchi 2012-10-17 17:03:17 UTC 
Boxes doesn't currently seem to set use-layout-size; I tested this with Documents (which sets the flag) and it appears to still work correctly. Code looks fine to me.
Comment 3 Emmanuele Bassi (:ebassi) 2012-10-17 17:53:56 UTC 
coolio, thanks for testing Cosimo.
Comment 4 Emmanuele Bassi (:ebassi) 2012-10-17 17:54:52 UTC 
Review of attachment 224472 [details] [review]:

looks good to me.
Comment 5 Emmanuele Bassi (:ebassi) 2012-10-17 22:09:46 UTC 
Attachment 224472 [details] pushed to master, and it's in the 1.4.0 release as well.