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 792503 - Compat tab warning / spacing
Compat tab warning / spacing
Status: RESOLVED FIXED
Product: gnome-terminal
Classification: Core
Component: Profiles
git master
Other Linux
: Normal minor
: ---
Assigned To: GNOME Terminal Maintainers
GNOME Terminal Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-01-13 23:00 UTC by Egmont Koblinger
Modified: 2018-01-13 23:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
restore warning (1.16 KB, patch)
2018-01-13 23:00 UTC, Egmont Koblinger
none Details | Review
remove warning; convert to a single gtkgrid (4.57 KB, patch)
2018-01-13 23:28 UTC, Egmont Koblinger
committed Details | Review

Description Egmont Koblinger 2018-01-13 23:00:19 UTC
Created attachment 366784 [details] [review]
restore warning

On Profile prefs -> Compatibility, the spacing at the top of the notebook page is about twice as big as on other tabs.

This is because there's a zero-height text label there which (probably accidentally) got lost from the UI several years ago, but is still translatable/translated.

Here's a patch to resurrect that text. Note that I don't understand what I'm doing with that max_width_chars=0, that's just how I managed to wrap the text.

Shall we restore this warning, or drop for good?

This is probably the only piece of italic text on the entire g-t UI. Keep it that way?

In the mean time, there's a vbox containing three children: this label, the grid of 4 rows, and the reset button. I guess this could just be a single grid of 5-6 rows.
Comment 1 Christian Persch 2018-01-13 23:07:21 UTC
(In reply to Egmont Koblinger from comment #0)
> Created attachment 366784 [details] [review] [review]
> restore warning
> 
> On Profile prefs -> Compatibility, the spacing at the top of the notebook
> page is about twice as big as on other tabs.
> 
> This is because there's a zero-height text label there which (probably
> accidentally) got lost from the UI several years ago, but is still
> translatable/translated.

Seems this was due to a glade bug (commit d99a84c45fce74b8cf2be0eab86185c79ca5c4e5) and went unnoticed.

> Here's a patch to resurrect that text. Note that I don't understand what I'm
> doing with that max_width_chars=0, that's just how I managed to wrap the
> text.
> 
> Shall we restore this warning, or drop for good?

Since it's been years with this gone without anyone missing it, we should drop it for good.

> In the mean time, there's a vbox containing three children: this label, the
> grid of 4 rows, and the reset button. I guess this could just be a single
> grid of 5-6 rows.

Yes.
Comment 2 Egmont Koblinger 2018-01-13 23:28:56 UTC
Created attachment 366785 [details] [review]
remove warning; convert to a single gtkgrid

Note that the "Reset" button became closer to the Narrow/Wide row. Meh. I restore the bigger gap if you insist :)
Comment 3 Egmont Koblinger 2018-01-13 23:29:41 UTC
Oh, and of course I'll fix the xml's indentation before submitting.
Comment 4 Christian Persch 2018-01-13 23:39:11 UTC
Comment on attachment 366785 [details] [review]
remove warning; convert to a single gtkgrid

> Note that the "Reset" button became closer to the Narrow/Wide row. Meh. I
> restore the bigger gap if you insist :)

I won't :-)