From kfm-devel  Thu Jul 23 16:02:30 2009
From: Andrea Iacovitti <aiacovitti () libero ! it>
Date: Thu, 23 Jul 2009 16:02:30 +0000
To: kfm-devel
Subject: Patch for bug #200795
Message-Id: <200907231802.30457.aiacovitti () libero ! it>
X-MARC-Message: https://marc.info/?l=kfm-devel&m=125995931226782
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--Boundary-00=_WmIaKn+hszacuGi"

--Boundary-00=_WmIaKn+hszacuGi
Content-Transfer-Encoding: 7bit
Content-Type: text/plain;
  charset="us-ascii"
Content-Disposition: inline

Hi,
i wrote a small patch that solves http://bugs.kde.org/show_bug.cgi?id=200795
In that case khtml don't honour 'border:none' for input text widget and native 
borders are drawn around.
Is the patch ok?

Thank you.

Regards,
Andrea.

--Boundary-00=_WmIaKn+hszacuGi
Content-Transfer-Encoding: 7bit
Content-Type: text/x-diff;
  charset="iso 8859-15";
  name="border-none-patch.diff"
Content-Disposition: attachment;
	filename="border-none-patch.diff"

--- khtml/rendering/render_replaced.cpp	2009-04-05 21:59:45.000000000 +0200
+++ khtml/rendering/render_replaced.cpp	2009-07-23 14:52:17.000000000 +0200
@@ -479,7 +479,13 @@
         }
 
         // Border:
-        if (shouldPaintBorder())
+
+        bool borderNone = (style()->borderLeftStyle() == BNONE ||
+                           style()->borderRightStyle()  == BNONE ||
+                           style()->borderTopStyle()    == BNONE ||
+                           style()->borderBottomStyle() == BNONE);
+
+        if (shouldPaintBorder() || borderNone)
         {
             if (QFrame* frame = qobject_cast<QFrame*>(m_widget))
                 frame->setFrameShape(QFrame::NoFrame);

--Boundary-00=_WmIaKn+hszacuGi--