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

Update CSSProperties.json for text-align and text-justify #14563

Merged
merged 3 commits into from
Oct 16, 2018

Conversation

narayani28
Copy link
Collaborator

@narayani28 narayani28 commented Oct 10, 2018

Adding css3 new values start, end, match-parent, justify-all to text-align property and adding the property text-justify with its values.
https://www.w3.org/TR/css-text-3/

@@ -209,6 +209,7 @@
"text-emphasis-position": {"values": ["above", "below", "left", "right"]},
"text-emphasis-style": {"values": ["circle", "dot", "double-circle", "filled", "none", "open", "sesame", "triangle"]},
"text-indent": {"values": ["inherit"]},
"text-justify": {"values": ["auto", "none", "inter-word", "inter-character"]},
Copy link
Collaborator

Choose a reason for hiding this comment

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

we have "inherit" as an option for most of the properties. maybe we can add here as well since it is applicable.

Copy link
Collaborator

@vickramdhawal vickramdhawal left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@swmitra swmitra merged commit 488edaf into adobe:master Oct 16, 2018
ficristo pushed a commit to quadre-code/quadre that referenced this pull request Oct 16, 2018
* Adding css3 new values start, end, match-parent, justify-all to text-align property and adding the property text-justify with its values.

* Update CSSProperties.json

* Adding "inherit" value for text-justify
ficristo pushed a commit to quadre-code/quadre that referenced this pull request Oct 16, 2018
* Adding css3 new values start, end, match-parent, justify-all to text-align property and adding the property text-justify with its values.

* Update CSSProperties.json

* Adding "inherit" value for text-justify
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