Bug 82367 – Rename CSS Exclusions "rect" value to "rectangle"

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

[Bear Travis]

The spec draft http://dev.w3.org/csswg/css3-exclusions/
has renamed the polygon css value "rect" to "rectangle"

[Hans Muller]

Created attachment 134426 [details]
Patch

[WebKit Review Bot]

Attachment 134426 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1
LayoutTests/ChangeLog:10:  Line contains tab character.  [whitespace/tab] [5]
Total errors found: 1 in 10 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Hans Muller]

Created attachment 134429 [details]
Patch

[Hans Muller]

Removed a tab character.

[Ryosuke Niwa]

Comment on attachment 134429 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=134429&action=review

> Source/WebCore/ChangeLog:11
> +        Changed the CSS exclusion shape called "rect" to "rectangle" to match
> +        the current draft spec - http://dev.w3.org/csswg/css3-exclusions.
> +
> +        No new tests were created; just updated the existing ones.

It would be helpful if you could refer to mailing list or whatever page where this change was discussed.

[Hans Muller]

(In reply to comment #5)
> ...
> It would be helpful if you could refer to mailing list or whatever page where this change was discussed.

Don't know if this point was discussed on a public forum, looking into that now...

[Hans Muller]

(In reply to comment #5)
> (From update of attachment 134429 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=134429&action=review
> 
> > Source/WebCore/ChangeLog:11
> > +        Changed the CSS exclusion shape called "rect" to "rectangle" to match
> > +        the current draft spec - http://dev.w3.org/csswg/css3-exclusions.
> > +
> > +        No new tests were created; just updated the existing ones.
> 
> It would be helpful if you could refer to mailing list or whatever page where this change was discussed.

Vincent Hardy provided the following references for this change.  Assuming that they're sufficient, please add this change to to the commit queue.  

[vh] See http://wiki.csswg.org/spec/css3-exclusions#paris-f2f-february-2012.

This is where the proposed changes were discussed:

http://wiki.csswg.org/ideas/functional-notation

The meeting minutes are at:
http://lists.w3.org/Archives/Public/www-style/2012Feb/0325.html

and the resolution is:
"Resolved: Adopt the proposed changes to Exclusions therein except for the suggestion to unify rect() and rectangle()."

So this shows the group discussed rect v.s., rectangle and decided to go with rectangle() for exclusions.

[Ryosuke Niwa]

(In reply to comment #7)
> Vincent Hardy provided the following references for this change.  Assuming that they're sufficient, please add this change to to the commit queue.  
> 
> [vh] See http://wiki.csswg.org/spec/css3-exclusions#paris-f2f-february-2012.
> 
> This is where the proposed changes were discussed:
> 
> http://wiki.csswg.org/ideas/functional-notation
> 
> The meeting minutes are at:
> http://lists.w3.org/Archives/Public/www-style/2012Feb/0325.html
> 
> and the resolution is:
> "Resolved: Adopt the proposed changes to Exclusions therein except for the suggestion to unify rect() and rectangle()."
> 
> So this shows the group discussed rect v.s., rectangle and decided to go with rectangle() for exclusions.

Please include some of that info in the change log.

[Hans Muller]

Created attachment 135464 [details]
Patch

[WebKit Review Bot]

Comment on attachment 135464 [details]
Patch

Clearing flags on attachment: 135464

Committed r113328: <http://trac.webkit.org/changeset/113328>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.