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 685534 - Login field height changing
Login field height changing
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: lock-screen
3.6.x
Other Linux
: Normal minor
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
3.6.1
Depends on:
Blocks:
 
 
Reported: 2012-10-04 22:57 UTC by Crieloue Gilles
Modified: 2012-10-15 21:21 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
st-entry: Force a relayout if necessary (1.59 KB, patch)
2012-10-15 19:43 UTC, Florian Müllner 		committed Details | Review

Description Crieloue Gilles 2012-10-04 22:57:31 UTC 
When the first letter is typed in the login field, the height increase a little bit.
It feels weird. Is it done on purpose ?
Comment 1 Matthias Clasen 2012-10-12 10:40:55 UTC 
This seems like a nice little polish thing. Can we get it fixed for 3.6.1 ?
Comment 2 Florian Müllner 2012-10-15 19:43:19 UTC 
Created attachment 226501 [details] [review]
st-entry: Force a relayout if necessary

ClutterText will only queue a relayout after font changes if it has
any contents other than the empty string. As a result, its height
request may change after the first character has been entered. To
avoid this visual glitch, force a relayout on actual font changes.

The Clutter behavior doesn't make much sense to me here, but if we want this issue fixed in the stable release at this point, I guess we'll have to use a workaround.
Comment 3 Jasper St. Pierre (not reading bugmail) 2012-10-15 20:30:01 UTC 
Review of attachment 226501 [details] [review]:

Do we have a Clutter bug / patch for this anywhere? Fix looks good, anyway.
Comment 4 Florian Müllner 2012-10-15 21:02:19 UTC 
Attachment 226501 [details] pushed as 487749c - st-entry: Force a relayout if necessary
Comment 5 Jasper St. Pierre (not reading bugmail) 2012-10-15 21:21:57 UTC 
> Do we have a Clutter bug / patch for this anywhere?