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 726951 - ui: remove unnecessary size groups
ui: remove unnecessary size groups
Status: RESOLVED FIXED
Product: gnome-chess
Classification: Applications
Component: General
git master
Other All
: Normal normal
: ---
Assigned To: gnome-chess-maint
gnome-chess-maint
Depends on:
Blocks:
 
 
Reported: 2014-03-24 03:44 UTC by Michael Catanzaro
Modified: 2014-03-25 15:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ui: remove unnecessary size groups (1.50 KB, patch)
2014-03-24 03:44 UTC, Michael Catanzaro
committed Details | Review
The bottom size group can also expand the history control buttons.... (51.66 KB, image/png)
2014-03-24 04:04 UTC, Michael Catanzaro
  Details
Screenshot - gnome-chess in English (60.63 KB, image/png)
2014-03-24 16:01 UTC, Yosef Or Boczko
  Details
Screenshot - gnome-chess in Hebrew (58.89 KB, image/png)
2014-03-24 16:02 UTC, Yosef Or Boczko
  Details
Partially revert "remove unnecessary size groups" (1.12 KB, patch)
2014-03-25 00:48 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2014-03-24 03:44:58 UTC
The horizontal size group is exacerbating the horrible resizing
behavior as the clock ticks down. It's not the size group's fault that
this is broken, but the size group makes it 1634435x worse.

The vertical size group is simply unnecessary in 3.12. In 3.10 it is
needed for the header bar buttons to have the same height, but forces
the buttons into odd (non-square) shapes. Let's just remove it.
Comment 1 Michael Catanzaro 2014-03-24 03:44:59 UTC
Created attachment 272723 [details] [review]
ui: remove unnecessary size groups
Comment 2 Michael Catanzaro 2014-03-24 04:04:22 UTC
Created attachment 272726 [details]
The bottom size group can also expand the history control buttons....
Comment 3 Michael Catanzaro 2014-03-24 14:47:11 UTC
Attachment 272723 [details] pushed as 53fdcac - ui: remove unnecessary size groups
Comment 4 Yosef Or Boczko 2014-03-24 16:01:58 UTC
Created attachment 272792 [details]
Screenshot - gnome-chess in English

Please, revert this patch, it not right in my screen (my resolution is
900 × 1440).
Comment 5 Yosef Or Boczko 2014-03-24 16:02:51 UTC
Created attachment 272795 [details]
Screenshot - gnome-chess in Hebrew

it more clear there is some pixels between the image-buttons to the text-button.
Comment 6 Michael Catanzaro 2014-03-25 00:40:01 UTC
Aaaah, the New Game button, I missed that. Yes, agreed. The top size group was harmless anyway; I was mostly concerned about the one on the bottom. They just happened to be right next to each other.

P.S. Unrelated: Looking at that screenshot, I'm going to revert [1] as well, but only in gnome-3-12. It's expected that your default fonts all contain the ratio character (pick a better fixed-width font!) and I think this will only sting Arch users, but I don't want to break that in 3.12.1 when the colon was used in 3.12.0.

[1] https://git.gnome.org/browse/gnome-chess/commit/?h=gnome-3-12&id=da695c6cc43db4c52a4410aa2d8e743811cef949
Comment 7 Michael Catanzaro 2014-03-25 00:48:15 UTC
The following fix has been pushed:
4812c81 Partially revert "remove unnecessary size groups"
Comment 8 Michael Catanzaro 2014-03-25 00:48:17 UTC
Created attachment 272826 [details] [review]
Partially revert "remove unnecessary size groups"

The header bar size group was actually important: it was keeping the New
Game button at the proper height. And since it wasn't causing any
problems, let's just put it back.
Comment 9 Yosef Or Boczko 2014-03-25 10:56:54 UTC
(In reply to comment #6)
> Aaaah, the New Game button, I missed that. Yes, agreed. The top size group was
> harmless anyway; I was mostly concerned about the one on the bottom. They just
> happened to be right next to each other.
> 
> P.S. Unrelated: Looking at that screenshot, I'm going to revert [1] as well,
> but only in gnome-3-12. It's expected that your default fonts all contain the
> ratio character (pick a better fixed-width font!) and I think this will only
> sting Arch users, but I don't want to break that in 3.12.1 when the colon was
> used in 3.12.0.
> 

I use Canteral, if you wonder what the font I use.

> [1]
> https://git.gnome.org/browse/gnome-chess/commit/?h=gnome-3-12&id=da695c6cc43db4c52a4410aa2d8e743811cef949
Comment 10 Michael Catanzaro 2014-03-25 14:11:11 UTC
The time labels use the "fixed" font:

$ fc-match fixed
DejaVuSans.ttf: "DejaVu Sans" "Book"

IIRC Arch's default "fixed" is an ancient bitmap font without Unicode coverage.
Comment 11 Yosef Or Boczko 2014-03-25 14:14:08 UTC
I wonder what this:
$ fc-match fixed
7x13.pcf.gz: "Fixed" "Regular"

I'm use Infinality-bundle fonts (freetype2-infinality, fontconfig-infinality and
cairo-infinality) and the default fonts of gnome settings. this not what in
default in Arch (see https://wiki.archlinux.org/index.php/Infinality-bundle%2Bfonts).
Comment 12 Michael Catanzaro 2014-03-25 15:42:32 UTC
You can try: https://wiki.archlinux.org/index.php/Font_Configuration#Disable_bitmap_fonts

Anyway, we've ventured quite off topic, sorry about that.