Bug 364426

Summary: Right margin too small in KPageDialog
Product: [Plasma] Breeze Reporter: Elvis Angelaccio <elvis.angelaccio>
Component: QStyleAssignee: Hugo Pereira Da Costa <hugo.pereira.da.costa>
Status: RESOLVED FIXED    
Severity: normal CC: cfeck, hugo.pereira.da.costa, kdelibs-bugs, simonandric5
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: http://commits.kde.org/oxygen/7d989d35643adfcd987560b9d3266b76edbf8246 Version Fixed In:
Attachments: Test case screenshot

Description Elvis Angelaccio 2016-06-17 17:48:54 UTC 
This is possibly a bug in KPageWidget or KPageView, but it's easier to reproduce in a KPageDialog.

Reproducible: Always

Steps to Reproduce:
1. build kwidgetsaddons (commit 9e86778676)
2. run $BUILD_DIR/tests/kpagedialogtest

Actual Results:  
The last character in the long string is cropped.

Expected Results:  
The long string should not be cropped.
Comment 1 Elvis Angelaccio 2016-06-17 17:49:59 UTC 
Created attachment 99551 [details]
Test case screenshot
Comment 2 Christoph Feck 2016-06-17 23:31:38 UTC 
I tested with the following widget styles:
- Fusion: works okey
- Windows: works okey
- Skulpture: works okey
- Oxygen: label is clipped
- Breeze: label is clipped

Unless Hugo confirms the bug is indeed in KPageDialog classes, I assume the bug is in the KDE widget styles.
Comment 3 Hugo Pereira Da Costa 2016-06-18 09:04:54 UTC 
Confirmed. will investigate. Must be something inconsistent in some sizeFromContent calculations ...
Comment 4 Hugo Pereira Da Costa 2016-07-20 14:26:46 UTC 
Git commit 4fb5a3bf8b1abcfcdabcf8a9c88276a44b8c2a31 by Hugo Pereira Da Costa.
Committed on 20/07/2016 at 14:17.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Fixed tabwidget size calculation from contents

M  +2    -17   kstyle/breezestyle.cpp

http://commits.kde.org/breeze/4fb5a3bf8b1abcfcdabcf8a9c88276a44b8c2a31
Comment 5 Hugo Pereira Da Costa 2016-07-20 14:59:36 UTC 
Git commit 7d989d35643adfcd987560b9d3266b76edbf8246 by Hugo Pereira Da Costa.
Committed on 20/07/2016 at 14:57.
Pushed by hpereiradacosta into branch 'Plasma/5.7'.

Fixed tabwidget size calculation from contents

M  +1    -15   kstyle/oxygenstyle.cpp

http://commits.kde.org/oxygen/7d989d35643adfcd987560b9d3266b76edbf8246