Bug 36713 – Support box-sizing without the vendor prefix

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Erik Arvidsson]

IE8 and Opera both support box-sizing without the vender prefix. So should we.

Also see the Mozilla bug where Anne van Kesteren writes:

  "CSS3 UI has reached CR. This means we can (and are encouraged to) drop the -moz- prefix from the 'box-sizing' property."

https://bugzilla.mozilla.org/show_bug.cgi?id=243412

[Peter Beverloo]

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.

[Darin Adler]

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?

[Erik Arvidsson]

Peter, we need to keep -webkit-box-sizing working for now since there is already a lot of web content out there using this.

[Peter Beverloo]

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?

[Erik Arvidsson]

Supporting both properties is the right thing to do.

I think updating the tests is the right thing to do as well.

[j.j.]

(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!

[Erik Arvidsson]

Created attachment 71612 [details]
Patch

[Erik Arvidsson]

Peter, I updated your patch to support both -webkit-box-sizing and box-sizing.

[Peter Beverloo]

Great, thank you.

[Dimitri Glazkov (Google)]

(In reply to comment #9)
> Great, thank you.

The patch is fine. Should we also put Peter in the creator line?

[Erik Arvidsson]

Created attachment 72841 [details]
Patch

[Dimitri Glazkov (Google)]

Comment on attachment 72841 [details]
Patch

ok.

[WebKit Commit Bot]

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

[Erik Arvidsson]

Created attachment 72966 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 72966 [details]
Patch for landing

Clearing flags on attachment: 72966

Committed r71348: <http://trac.webkit.org/changeset/71348>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Simon Fraser (smfr)]

Why were none of the CSS regulars (me, dethbakin, mitz, hyatt) copied on this bug?

[Andy Estes]

<rdar://problem/7867854>