Bug 340341 - Contents of a tool button clipped with Breeze style
Summary: Contents of a tool button clipped with Breeze style
Status: CLOSED FIXED
Alias: None
Product: Breeze
Classification: Plasma
Component: QStyle (other bugs)
Version First Reported In: 5.1.0
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Hugo Pereira Da Costa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-25 19:52 UTC by Jarosław Staniek
Modified: 2014-11-23 23:08 UTC (History)
1 user (show)

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


Attachments
Clipped tool button in Designer's property editor (4.71 KB, image/png)
2014-10-25 19:52 UTC, Jarosław Staniek
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jarosław Staniek 2014-10-25 19:52:02 UTC
Contents of a tool button clipped with Breeze style (Qt 4).

To test, run Qt 4 Designer and in the Property Editor select any property that has a [...] button.
In other styles all three dots appear. In Breeze only one dot, the rest is clipped.

Expected Results:  
Entire content is displayed.
Comment 1 Jarosław Staniek 2014-10-25 19:52:57 UTC
Created attachment 89321 [details]
Clipped tool button in Designer's property editor
Comment 2 Hugo Pereira Da Costa 2014-10-26 17:47:04 UTC
Git commit 648e46da778a7a7ff23ecc371fb1193c9aa54079 by Hugo Pereira Da Costa.
Committed on 26/10/2014 at 17:45.
Pushed by hpereiradacosta into branch 'Plasma/5.1'.

Do not takeout margins from toolbutton before rendering text, unless they have inline indicator, because
some custom widgets do not respect sizeFromContents

M  +3    -7    kstyle/breezestyle.cpp

http://commits.kde.org/breeze/648e46da778a7a7ff23ecc371fb1193c9aa54079
Comment 3 Hugo Pereira Da Costa 2014-10-26 17:47:05 UTC
Git commit e39b2c94710cbc7b90c9605baeeda83793fba33d by Hugo Pereira Da Costa.
Committed on 26/10/2014 at 17:45.
Pushed by hpereiradacosta into branch 'master'.

Do not takeout margins from toolbutton before rendering text, unless they have inline indicator, because
some custom widgets do not respect sizeFromContents

M  +3    -7    kstyle/breezestyle.cpp

http://commits.kde.org/breeze/e39b2c94710cbc7b90c9605baeeda83793fba33d