Commit 94ea668f authored by Marco Martin's avatar Marco Martin

be less aggressive when to disable borders

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

Reviewers: #plasma, hpereiradacosta

Reviewed By: hpereiradacosta

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D8160
parent 98dceb03
......@@ -3267,7 +3267,7 @@ namespace Oxygen
const QPalette& palette( option->palette );
// make sure there is enough room to render frame
if( rect.height() <= 2*Metrics::LineEdit_FrameWidth + option->fontMetrics.height() )
if( rect.height() < 2*Metrics::LineEdit_FrameWidth + option->fontMetrics.height() )
{
const QColor background( palette.color( QPalette::Base ) );
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment