Skip to content

Conversation

@nhoizey
Copy link
Contributor

@nhoizey nhoizey commented Apr 1, 2019

It looks like 70e40c9 introduced the text-decoration-skip-ink property, but kept the example from the previous text-decoration-skip one.

It looks like 70e40c9 introduced the `text-decoration-skip-ink` property, but kept the example from the previous `text-decoration-skip` one.
@nhoizey
Copy link
Contributor Author

nhoizey commented Apr 1, 2019

Aformentioned commit was from @kojiishi

@dontcallmedom
Copy link
Member

Marked as non substantive for IPR from ash-nazg.

@nhoizey
Copy link
Contributor Author

nhoizey commented Apr 1, 2019

@dontcallmedom sorry, I'm not used to PRs here, should I have open an issue instead?

@dontcallmedom
Copy link
Member

@nhoizey no worry at all, your pull request looks more than appropriate (although I'll leave it to the editors to do the real assessment :) I was mostly using one of our tools to indicate your pull request is acceptable from an IPR perspective

@nhoizey
Copy link
Contributor Author

nhoizey commented Apr 1, 2019

@dontcallmedom ok, thanks!

Copy link
Contributor

@kojiishi kojiishi left a comment

Choose a reason for hiding this comment

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

Thank you for the PR!

@kojiishi kojiishi merged commit b1ad4fb into w3c:master Apr 1, 2019
@nhoizey
Copy link
Contributor Author

nhoizey commented Apr 1, 2019

@kojiishi you're welcome!

@nhoizey nhoizey deleted the patch-1 branch April 1, 2019 15:32
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.

3 participants