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 778272 - batch-rename: scrolled window height change
batch-rename: scrolled window height change
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
master
Other Linux
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-02-07 10:25 UTC by Alexandru Pandelea
Modified: 2017-03-03 16:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
batch-rename: fix stack child height (1.37 KB, patch)
2017-02-07 10:29 UTC, Alexandru Pandelea
none Details | Review
batch-rename: fix stack child height (1.48 KB, patch)
2017-02-07 19:36 UTC, Alexandru Pandelea
committed Details | Review

Description Alexandru Pandelea 2017-02-07 10:25:44 UTC
See patch.
Comment 1 Alexandru Pandelea 2017-02-07 10:29:12 UTC
Created attachment 345091 [details] [review]
batch-rename: fix stack child height

With numbering added, the format mode stack child is slightly smaller
than the replace one, so the scrolled window height is slightly
changing when switching between modes.

To fix this, add a top margin to the numbering box, so that the two
stack children have the same size.
Comment 2 Carlos Soriano 2017-02-07 12:24:47 UTC
Review of attachment 345091 [details] [review]:

Well spotted!

This is not a good fix though, calibrating UI by pixel spacing is going to break any time (even changing the font). You need to rely on the code, making the two children the same size, regarding what size it is. For that you know the general solution, GtkSizeGroup.

However, there is also some general solution for some containers, which is called "homogeneus". Check the homogeneus property of the stack, and you will fix the problem :)
Comment 3 Alexandru Pandelea 2017-02-07 19:36:24 UTC
Created attachment 345139 [details] [review]
batch-rename: fix stack child height

With numbering added, the format mode stack child is slightly smaller
than the replace one, so the scrolled window height is slightly
changing when switching between modes.

The problem is that the row spacing in the replace stack child is
greater by 1px than the spacing in the format stack child.

To fix this make the replace stack child have the same row spacing.
Comment 4 Carlos Soriano 2017-03-03 12:38:29 UTC
Review of attachment 345139 [details] [review]:

yeah!
Comment 5 Carlos Soriano 2017-03-03 16:07:41 UTC
Attachment 345139 [details] pushed as 76cefc8 - batch-rename: fix stack child height