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 720188 - Panel is too wide
Panel is too wide
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: Online Accounts
git master
Other All
: Normal normal
: ---
Assigned To: GNOME Online Accounts maintainer(s)
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-12-10 12:07 UTC by Andreas Nilsson
Modified: 2014-03-03 11:06 UTC
See Also:
GNOME target: 3.12
GNOME version: ---


Attachments
screenshot (475.94 KB, image/png)
2013-12-10 12:07 UTC, Andreas Nilsson
  Details
online-accounts: Set max width in chars for label (1.27 KB, patch)
2014-03-02 14:43 UTC, Volker Sobek (weld)
committed Details | Review

Description Andreas Nilsson 2013-12-10 12:07:24 UTC
Created attachment 263911 [details]
screenshot

See screenshot
Using continuous
Comment 1 Benjamin Otte (Company) 2013-12-11 16:55:30 UTC
This is fallout from the change where we now give windows their natural size or the screen size and no longer their minimum size or 640px.

Unfortunately, wrapping labels prefer to not be wrapped and will therefore request the full text's width as the natural size. So if you have a label that says "Adding an account allows your application to access it for documents, mail, contacts, calendar, chat and more" for the case where no account exists and that label is still used for size computation of a GtkStack - that currently shows configuration options for the current account - than that stack will request a rather large natural size.

And that is why this dialog is so wide.
Comment 2 Volker Sobek (weld) 2014-03-02 14:43:00 UTC
Created attachment 270699 [details] [review]
online-accounts: Set max width in chars for label

Set a limit of 40 chars to the label explaining the accounts panel when
no accounts have been added yet. This prevents the label from expanding
itself and the whole window arbitrarily.
Comment 3 Debarshi Ray 2014-03-03 09:53:05 UTC
Review of attachment 270699 [details] [review]:

Looks good. Thanks for the patch.

::: panels/online-accounts/online-accounts.ui
@@ +147,3 @@
                         <property name="justify">center</property>
                         <property name="wrap">True</property>
+                        <property name="max_width_chars">40</property>

I swear I had tried this when the bug was first reported, and it still turned out wide. Maybe I was stupid.