Bug 370281

Summary: Lock screen notification label sometimes wraps over when it shouldn't
Product: [Plasma] plasmashell Reporter: Pritam Baral <pritam>
Component: Global Theme packagesAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: plasma-bugs-null, pritam
Priority: NOR    
Version First Reported In: 5.8.0   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
Latest Commit: http://commits.kde.org/plasma-workspace/e26c52414600c34078445aa2b2d3e450a2a44a6b Version Fixed/Implemented In:
Sentry Crash Report:

Description Pritam Baral 2016-10-08 04:39:31 UTC 
When I feed the wrong password, the "Unlocking failed" message wraps over to a new line while there is plenty of space in the first line itself. The text is also left-aligned instead of being center-aligned, which gives it a bad appearance.

The behaviour doesn't happen on all fonts and font sizes. My system uses Oxygen at size 8 with a forced DPI of 120 (it's a 12.5" screen). If I change any of these three factors, it renders okay. On some rare conditions, it wraps over.

Reproducible: Sometimes

Steps to Reproduce:
1. Use a 1366x768 resolution at a forced font DPI of 120
2. Use Oxygen as the system font at a size of 8
3. Feed the wrong password at the lock screen

Actual Results:  
As shown in this image: http://imgur.com/Iw0qJzT

Expected Results:  
The text message "Unlocking failed" should appear in one line, since there's plenty of room there, and should be center-aligned.
Comment 1 Pritam Baral 2016-10-08 04:57:28 UTC 
Patch submitted for review: https://phabricator.kde.org/D2984
Comment 2 David Edmundson 2016-10-09 23:14:42 UTC 
Git commit e26c52414600c34078445aa2b2d3e450a2a44a6b by David Edmundson, on behalf of Pritam Baral.
Committed on 09/10/2016 at 23:14.
Pushed by davidedmundson into branch 'Plasma/5.8'.

Center the notification label on Breeze's lockscreen

and make it use the full width before wrapping

Test Plan:
Tested on the same system that reported the bug; works for all
combinations of font, font size and DPI tested.

Reviewers: #plasma, davidedmundson

Reviewed By: #plasma, davidedmundson

Subscribers: davidedmundson, jensreuterberg, pritambaral, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D2984

M  +2    -0    lookandfeel/contents/components/SessionManagementScreen.qml

http://commits.kde.org/plasma-workspace/e26c52414600c34078445aa2b2d3e450a2a44a6b