Bug 364426 - Right margin too small in KPageDialog
Summary: Right margin too small in KPageDialog
Status: RESOLVED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: QStyle (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-17 17:48 UTC by Elvis Angelaccio
Modified: 2016-07-20 14:59 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Test case screenshot (18.23 KB, image/png)
2016-06-17 17:49 UTC, Elvis Angelaccio
Details

Note You need to log in before you can comment on or make changes to this bug.
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