Summary: | left/right padding not applied to input text | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Andrea Iacovitti <aiacovitti> |
Component: | khtml forms | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | 4.7.3 | ||
Target Milestone: | --- | ||
Platform: | Unlisted Binaries | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/kdelibs/658759cb7c436af133b770bc50c99219dff10814 | Version Fixed In: | 4.8.3 |
Sentry Crash Report: | |||
Attachments: | testcase |
Git commit dee2801c0a86fec1c642da9710b17b0a7bec3ecd by Andrea Iacovitti. Committed on 20/04/2012 at 08:18. Pushed by aiacovitti into branch 'KDE/4.8'. Fix rendering of text input elements having padding by removing the use of cached parent's proxy style, that end up to return a wrong content rect size, and using ad-hoc style() function instead. Actually only KLineEdit inherits KdeUiProxyStyle (since kde 4.5) so the behavior of our internal proxy is unchanged for the other form elements (i.e m_proxy was always 0 for them). Also the crash reported in #197270 is no longer reproducible after this change, likely it has been fixed elsewhere... Related: bug 280226 FIXED-IN: 4.8.3 M +7 -11 khtml/rendering/render_form.cpp http://commits.kde.org/kdelibs/dee2801c0a86fec1c642da9710b17b0a7bec3ecd Git commit 658759cb7c436af133b770bc50c99219dff10814 by Ivan Čukić, on behalf of Andrea Iacovitti. Committed on 20/04/2012 at 08:18. Pushed by ivan into branch 'ivan/plasma-active-patches'. Fix rendering of text input elements having padding by removing the use of cached parent's proxy style, that end up to return a wrong content rect size, and using ad-hoc style() function instead. Actually only KLineEdit inherits KdeUiProxyStyle (since kde 4.5) so the behavior of our internal proxy is unchanged for the other form elements (i.e m_proxy was always 0 for them). Also the crash reported in #197270 is no longer reproducible after this change, likely it has been fixed elsewhere... Related: bug 280226 FIXED-IN: 4.8.3 M +7 -11 khtml/rendering/render_form.cpp http://commits.kde.org/kdelibs/658759cb7c436af133b770bc50c99219dff10814 |
Created attachment 62987 [details] testcase Version: 4.7.0 OS: Linux khtml doesn't render input text with specified left/right padding Reproducible: Always Steps to Reproduce: Load the testcase Expected Results: :) It seems KHTMLProxyStyle is broken here (patch): +++ render_form.cpp @@ -101,7 +101,7 @@ SubElement element, const QStyleOption *option, const QWidget *widget ) const { - QRect r = proxy()->subElementRect(element, option, widget); + QRect r = style()->subElementRect(element, option, widget); switch (element) { case QStyle::SE_PushButtonContents: case QStyle::SE_LineEditContents: