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 340722 - GtkToolbar::max-child-expand style property
GtkToolbar::max-child-expand style property
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkToolbar
unspecified
Other All
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2006-05-05 10:38 UTC by Michael Natterer
Modified: 2011-02-04 16:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch implementing the above (2.91 KB, patch)
2006-05-05 10:39 UTC, Michael Natterer
none Details | Review

Description Michael Natterer 2006-05-05 10:38:46 UTC
Attach patch from maemo-gtk adds a "max-child-expand" child property
which limits the expanding of expanding tool items. It's pretty useful
when there are only a few items that should expand if possible, but should
not take all available space, because they would look too distributed then.
Comment 1 Michael Natterer 2006-05-05 10:39:44 UTC
Created attachment 64851 [details] [review]
Patch implementing the above
Comment 2 Soren Sandmann Pedersen 2006-05-05 14:30:32 UTC
So what you want is really an item with a natural size, that is allowed to shrink, no?

In any case, this patch doesn't look quite correct to me. For example in the case where there is both a fully-expandable and an expandable-to-a-point item, it looks to me like if the available extra space is more than twice the max-child-expand width, the fully-expandable item will not be assigned all of the remaining space.
Comment 3 Michael Natterer 2006-05-05 14:36:24 UTC
Err, i didn't mean to say "child property". The property affects all
expanding children, so there can't be a mix of fully-expandable and
expandable-to-a-point.
Comment 4 Tim Janik 2006-05-16 11:08:55 UTC
ok, the patch looks good and also makes sense to me. i just think that explicitely defining DEFAULT_MAX_CHILD_EXPAND tends to uneccessarily spread out the property definition, so should be omitted.
as i understood, Søren didn't really object aginst the property as a style property, which means the patch can go in now.
Comment 5 Michael Natterer 2006-05-16 13:52:03 UTC
Fixed in CVS:

2006-05-16  Michael Natterer  <mitch@imendio.com>

	* gtk/gtktoolbar.c: added "max-child-expand" style property which
	limits the space taken by expanding tool items. Fixes bug #340722.