Summary: | Support box-sizing without the vendor prefix | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Erik Arvidsson <arv> | ||||||||||
Component: | CSS | Assignee: | Erik Arvidsson <arv> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | aestes, commit-queue, dglazkov, moz, peter, simon.fraser | ||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Attachments: |
|
Description
Erik Arvidsson
2010-03-27 17:17:54 PDT
Created attachment 62420 [details]
Patch
Renamed -webkit-box-sizing to box-sizing and updated all relevant tests/default UA styling/Web Inspector.
While continuing to allow people to use -webkit-box-sizing (e.g. supporting them both) wouldn't be hard to add, I decided to rename it because the Basic UI Module is a CR and it matches support of other browsers:
Internet Explorer 8+: box-sizing
Opera 7+: box-sizing
Konqueror 3.3.2+: box-sizing
Mozilla Firefox: -moz-box-sizing
WebKit (current): -webkit-box-sizing
Mozilla prefixed their property because it also supports "padding-box", where the spec doesn't include it.
Comment on attachment 62420 [details]
Patch
Adding plain old "box-sizing" requires only that the standard is far enough along that we are confident we will not create incompatibilities with other browsers. But removing "-webkit-box-sizing" requires research to determine what content is using the webkit-prefixed name. Please don't remove it without doing the research!
Haven’t we already gone over this in the context of -webkit-opacity?
Peter, we need to keep -webkit-box-sizing working for now since there is already a lot of web content out there using this. Ok -- I understand that I'm taking this too lightly and won't be doing so again, sorry. It takes some time to get accustomed to the project. It would help if you could give me an indication about how to do the research, as I'm presuming checking the, say, 10.000 largest websites for usage of properties won't do. Can I modify the patch to support both properties? If so, would it be preferred to submit the modified test-cases as I've done with this one? Supporting both properties is the right thing to do. I think updating the tests is the right thing to do as well. (In reply to comment #1) > Mozilla prefixed their property because it also supports "padding-box", where > the spec doesn't include it. Reading the relevant Mozilla bug, I think the concerns are bugs in Gecko, lack of testcases and spec issues. See also https://developer.mozilla.org/en/CSS/box-sizing#Notes That said, please make people's life easier and fix it! Created attachment 71612 [details]
Patch
Peter, I updated your patch to support both -webkit-box-sizing and box-sizing. Great, thank you. (In reply to comment #9) > Great, thank you. The patch is fine. Should we also put Peter in the creator line? Created attachment 72841 [details]
Patch
Comment on attachment 72841 [details]
Patch
ok.
Comment on attachment 72841 [details] Patch Rejecting patch 72841 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'apply-attachment', '--force-clean', '--non-interactive', 72841]" exit_code: 2 Last 500 characters of output: ebCore/css/CSSPropertyNames.in patching file WebCore/css/CSSStyleSelector.cpp Hunk #1 succeeded at 4971 (offset 1 line). patching file WebCore/css/quirks.css patching file WebCore/css/view-source.css patching file WebCore/css/wml.css patching file WebKitTools/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file WebKitTools/iExploder/htdocs/cssproperties.in Failed to run "[u'/Projects/CommitQueue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Dimitri Glazkov', u'--force']" exit_code: 1 Full output: http://queues.webkit.org/results/5152007 Created attachment 72966 [details]
Patch for landing
Comment on attachment 72966 [details] Patch for landing Clearing flags on attachment: 72966 Committed r71348: <http://trac.webkit.org/changeset/71348> All reviewed patches have been landed. Closing bug. Why were none of the CSS regulars (me, dethbakin, mitz, hyatt) copied on this bug? |