D8160: ibe less aggressive when to disable borders
Marco Martin
noreply at phabricator.kde.org
Fri Oct 6 14:16:28 UTC 2017
mart created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
if the control height is
2*Metrics::LineEdit_FrameWidth + option->fontMetrics.height()
it means that "it just fits", therefore still draw the frame
the qqc2-desktop style uses that logic to get default line edit
heights, so they all got disabled by just one pixel
TEST PLAN
tested qwidget and qml apps
REPOSITORY
R113 Oxygen Theme
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D8160
AFFECTED FILES
kstyle/oxygenstyle.cpp
To: mart
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171006/eb599e39/attachment-0001.html>
More information about the Plasma-devel
mailing list