WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
35629
[Chromium] Should remove paddings of Win/Linux chromium's buttons
https://bugs.webkit.org/show_bug.cgi?id=35629
Summary
[Chromium] Should remove paddings of Win/Linux chromium's buttons
Shinichiro Hamaji
Reported
2010-03-02 22:18:09 PST
Per discussion on
Bug 35231
and
http://code.google.com/p/chromium/issues/detail?id=1437
, let's remove the extra paddings which causes web compatibility issues.
Attachments
Patch v1
(2.24 KB, patch)
2010-03-02 22:20 PST
,
Shinichiro Hamaji
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Shinichiro Hamaji
Comment 1
2010-03-02 22:20:21 PST
Created
attachment 49878
[details]
Patch v1
Shinichiro Hamaji
Comment 2
2010-03-02 22:22:30 PST
> Created an attachment (id=49878) [details] > Patch v1
Or, should we remove buttonInternalPadding* even from RenderTheme ? David, could you give me your suggestion if you have any opinion about this?
Shinichiro Hamaji
Comment 3
2010-03-02 22:25:28 PST
Comment on
attachment 49878
[details]
Patch v1 cq- because this patch will break bunch of chromium's layout tests so I need to communicate with WebKit sheriff before I land this patch.
Eric Seidel (no email)
Comment 4
2010-03-05 16:25:23 PST
Comment on
attachment 49878
[details]
Patch v1 OK.
Ojan Vafai
Comment 5
2010-03-05 16:55:35 PST
To give context for this. We've found a number of sites whose layout breaks due to this extra internal padding. We originally added this to match Firefox since padding:0 buttons look pretty crappy without having any internal padding. Chromium would like to run an experiment removing the padding and see if there's any fallout. We're hoping that not many sites use padding:0. If it turns out they do, then we'll probably revert this change. If they don't, we should consider making Safari Win also not having internal button padding. Some sites that break with the internal padding:
http://topkayaker.net/TopKayakerShop/
http://www.qotsa.com/
http://catalinaexpress.com/
http://www.dekalb.k12.ga.us/
http://balkanweb.com/poll/pollsys/
http://www.indianmasala.com/
Shinichiro Hamaji
Comment 6
2010-03-09 00:24:12 PST
Comment on
attachment 49878
[details]
Patch v1 Clearing flags on attachment: 49878 Committed
r55714
: <
http://trac.webkit.org/changeset/55714
>
Shinichiro Hamaji
Comment 7
2010-03-09 00:24:21 PST
All reviewed patches have been landed. Closing bug.
Erik Arvidsson
Comment 8
2010-03-26 17:36:03 PDT
***
Bug 31703
has been marked as a duplicate of this bug. ***
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug