Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Update CSS at-rule hints #14013

Merged
merged 4 commits into from
Jan 14, 2018
Merged

Update CSS at-rule hints #14013

merged 4 commits into from
Jan 14, 2018

Conversation

valtlai
Copy link
Contributor

@valtlai valtlai commented Jan 5, 2018

Add @counter-style and @font-feature-values at-rules.
Reorder the rules alphabetically.

Add @counter-style and @font-feature-values.
(Align colons.)
@valtlai valtlai changed the title Update CSS at-rules Update CSS at-rule hints Jan 5, 2018
"@charset": "Defines the character set used by the style sheet.",
"@counter-style": "Defines specific counter styles that are not part of the predefined set of styles.",
"@font-face": "Describes the aspect of an external font to be downloaded.",
"@font-feature-values": "Define common names in font-variant-alternates for feature activated differently in OpenType.",
Copy link
Collaborator

@petetnt petetnt Jan 8, 2018

Choose a reason for hiding this comment

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

MDN phrases this as Defines a common name in font-variant-alternates for features activated differently in OpenType., not quite sure which one is more correct though
¯\_(ツ)_/¯

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, these descriptions are from another MDN page.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Then it's good enough for me (maybe fix them in MDN too? 😂 )

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made it a bit more consistent by adding -s. 😇

Copy link
Collaborator

@ficristo ficristo left a comment

Choose a reason for hiding this comment

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

Fix the test.
Otherwise LGTM.

@valtlai
Copy link
Contributor Author

valtlai commented Jan 14, 2018

Fixed the test.

@ficristo ficristo merged commit cad7bd2 into adobe:master Jan 14, 2018
@ficristo
Copy link
Collaborator

Thank you!

ficristo pushed a commit to quadre-code/quadre that referenced this pull request Jan 15, 2018
* Update CSS at-rules

Add @counter-style and @font-feature-values.
(Align colons.)

* Revert colon aligment

* Make CSS at-rule descriptions bit more consistent

* Fix the unit test for CSS At-Rule Hints
ficristo pushed a commit to quadre-code/quadre that referenced this pull request Jan 15, 2018
* Update CSS at-rules

Add @counter-style and @font-feature-values.
(Align colons.)

* Revert colon aligment

* Make CSS at-rule descriptions bit more consistent

* Fix the unit test for CSS At-Rule Hints
@valtlai valtlai deleted the valtlai/css-at-rules branch April 18, 2018 19:56
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants