Skip to content

Conversation

@therealglazou
Copy link
Contributor

This PR is about https://lists.w3.org/Archives/Public/www-style/2016Mar/0235.html and is ready to merge if the CSSWG agrees on the change later today.


<div class=example>
The {{CSS/escape()}} method can also be used for escaping strings, although it escapes characters that don't strictly need to be
The {{CSS/escape()}} method can also be used for serializing URL fragments, although it escapes characters that don't strictly need to be
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here I meant strings as in CSS strings (as opposed to CSS idents)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, reverting that change then

@zcorpan
Copy link
Contributor

zcorpan commented Mar 16, 2016

LGTM. Do you want to squash?

@therealglazou therealglazou force-pushed the EscapedToIdentifierSerialization branch from bde0462 to df49d5f Compare March 16, 2016 12:15
@therealglazou
Copy link
Contributor Author

@zcorpan done

zcorpan pushed a commit that referenced this pull request Mar 16, 2016
@zcorpan
Copy link
Contributor

zcorpan commented Mar 16, 2016

Thanks, merged as 147461b

@zcorpan zcorpan closed this Mar 16, 2016
@therealglazou therealglazou deleted the EscapedToIdentifierSerialization branch March 17, 2016 09:42
birtles added a commit to birtles/csswg-drafts that referenced this pull request Dec 4, 2017
frivoal pushed a commit to frivoal/csswg-drafts that referenced this pull request Nov 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants