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
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.