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 658993 - Two GtkGrid spacing issues
Two GtkGrid spacing issues
Status: RESOLVED FIXED
Product: orca
Classification: Applications
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Joanmarie Diggs (IRC: joanie)
Orca Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-09-13 22:02 UTC by Joanmarie Diggs (IRC: joanie)
Modified: 2011-09-13 23:13 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix (2.02 KB, patch)
2011-09-13 22:05 UTC, Joanmarie Diggs (IRC: joanie)
committed Details | Review
screenshot (50.26 KB, image/png)
2011-09-13 22:14 UTC, Joanmarie Diggs (IRC: joanie)
  Details
screenshot 2 - what the gui is meant to look like (49.89 KB, image/png)
2011-09-13 22:32 UTC, Joanmarie Diggs (IRC: joanie)
  Details

Description Joanmarie Diggs (IRC: joanie) 2011-09-13 22:02:59 UTC
Orca's Preferences dialog was recently redone. As part of this change, all instances of GtkHBox, GtkVBox, GtkBox, and GtkTable were converted to use GtkGrid. In this process, I discovered that the GtkGrid row-spacing and column-spacing properties seemed to be backwards. Benjamin has since fixed this:
http://git.gnome.org/browse/gtk+/commit/?id=d717a2dcfc8603561f8a0f78982244e8b8fd9006

Unfortunately, there were two grid instances where I failed to ensure things would look as expected once the Gtk+ bug was fixed. And those now look really, really bad. :-/

See attached screenshot and weep.
Comment 1 Joanmarie Diggs (IRC: joanie) 2011-09-13 22:05:06 UTC
Created attachment 196440 [details] [review]
Fix

We're in GUI freeze so I need to get approval from the Release Team before I can commit this.
Comment 2 Joanmarie Diggs (IRC: joanie) 2011-09-13 22:14:42 UTC
Created attachment 196442 [details]
screenshot
Comment 3 Joanmarie Diggs (IRC: joanie) 2011-09-13 22:32:10 UTC
Created attachment 196443 [details]
screenshot 2 - what the gui is meant to look like

This screenshot was taken prior to the Gtk+ fix. And it's what the gui will once again look like should the fix be approved for freeze break.
Comment 5 Joanmarie Diggs (IRC: joanie) 2011-09-13 23:13:27 UTC
Approval: https://mail.gnome.org/archives/release-team/2011-September/msg00133.html

Doc Team announcement: https://mail.gnome.org/archives/gnome-doc-list/2011-September/msg00123.html

Closing as FIXED.