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 724742 - GtkRevealer not allocating enough space
GtkRevealer not allocating enough space
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: .General
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-02-19 18:36 UTC by Zeeshan Ali
Modified: 2014-02-20 01:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to Revealer that helps (4.28 KB, patch)
2014-02-19 18:36 UTC, Zeeshan Ali
none Details | Review
Screenshot of the issue in Boxes (281.79 KB, image/png)
2014-02-19 18:37 UTC, Zeeshan Ali
  Details

Description Zeeshan Ali 2014-02-19 18:36:18 UTC
Created attachment 269717 [details] [review]
Patch to Revealer that helps

After I ported Boxes to use GtkRevealer to reveal the sidebar in properties/wizard views instead of a custom ClutterActor, the sidebar shrank even though I set vexpand=true on all relevant widgets. The boxes change is here:

https://git.gnome.org/browse/gnome-boxes/log/?h=wip/less-actors2 (The one titled "sidebar: Derive from Gtk.Revealer").

Benjamin provided the attached patch to see if it helps and it does.
Comment 1 Zeeshan Ali 2014-02-19 18:37:37 UTC
Created attachment 269718 [details]
Screenshot of the issue in Boxes
Comment 2 Zeeshan Ali 2014-02-19 19:18:36 UTC
Turns out that the attached patch at least breaks the actionbar in selection mode that is revealed from bottom to upwards. The buttons on the actionbar are missing with this patch and its just an empty bar. Without the patch, its fine.
Comment 3 Benjamin Otte (Company) 2014-02-20 01:20:21 UTC
commit 118969e70a548c9b8f5b54d664aea7f4415e3fc5
Author: Benjamin Otte <otte@redhat.com>
Date:   Thu Feb 20 01:47:21 2014 +0100

    reftests: Add a reftest for the latest fix
    
    https://bugzilla.gnome.org/show_bug.cgi?id=724742

commit b9e37f8d2d4d14d3cb23f56a795fcdd2e6128e88
Author: Benjamin Otte <otte@redhat.com>
Date:   Wed Feb 19 21:15:25 2014 +0100

    revealer: Allocate extra size to child
    
    Do not constrain the child widget to its preferred size when the
    revealer is larger.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=724742