Bug 36413 - [Qt] RenderTheme: fix size adjustment for text fields
Summary: [Qt] RenderTheme: fix size adjustment for text fields
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Forms (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P3 Normal
Assignee: Nobody
URL:
Keywords: Qt
: 36030 (view as bug list)
Depends on:
Blocks: 35784
  Show dependency treegraph
 
Reported: 2010-03-20 10:07 PDT by Jakub Wieczorek
Modified: 2010-03-26 08:41 PDT (History)
4 users (show)

See Also:


Attachments
patch (7.91 KB, patch)
2010-03-20 10:17 PDT, Jakub Wieczorek
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Wieczorek 2010-03-20 10:07:40 PDT
RenderThemeQt.cpp "hardcodes" the preferred dimensions for text fields but in fact it is not necessary, because WebCore already does that (20 cols x 2 rows for text areas and 20 cols x 1 row for text fields). To match other ports the size shouldn't be altered by the render theme, I believe.

Moreover, the padding for text fields is not properly applied:
- RenderTheme should also change the horizontal padding.
- it should work the same for text areas - it does not now, which causes rendering issues with the Oxygen style.
Comment 1 Jakub Wieczorek 2010-03-20 10:17:19 PDT
Created attachment 51222 [details]
patch

With this change, 3 test results had to be updated but I'm not sure if there isn't anything left to update.
Comment 2 Giacomo 2010-03-20 14:13:37 PDT
Is this related to: https://bugs.webkit.org/show_bug.cgi?id=36030 ?
Comment 3 Simon Hausmann 2010-03-21 15:48:09 PDT
Tor Arne, could you take a look at this one?

Thanks :)
Comment 4 Eric Seidel (no email) 2010-03-25 01:50:24 PDT
Comment on attachment 51222 [details]
patch

OK.
Comment 5 Jakub Wieczorek 2010-03-25 10:30:55 PDT
Comment on attachment 51222 [details]
patch

Clearing flags on attachment: 51222

Committed r56555: <http://trac.webkit.org/changeset/56555>
Comment 6 Jakub Wieczorek 2010-03-25 10:31:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Giacomo 2010-03-25 12:19:12 PDT
*** Bug 36030 has been marked as a duplicate of this bug. ***
Comment 8 Simon Hausmann 2010-03-26 08:41:20 PDT
Revision r56555 cherry-picked into qtwebkit-2.0 with commit cd23c453d081a795f63d594e3da8b2b580726ab7