Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[motion] Editorial improvements to offset-distance #67

Merged
merged 1 commit into from
Nov 9, 2016
Merged

[motion] Editorial improvements to offset-distance #67

merged 1 commit into from
Nov 9, 2016

Conversation

fantasai
Copy link
Contributor

@fantasai fantasai commented Nov 5, 2016

  • Clarify property description and cross-link to 'offset-path' which it references.
  • Use a more precise description for the <> value. TODO: The word "anchor point" should later be hyperlinked to a instance under 'offset-anchor'.
  • Use em-dash instead of hyphen, per English typographic rules.
  • Use Bikeshed cross-reference syntax.

@fantasai
Editorial improvements to offset-distance
562b7f8 
* Clarify property description and cross-link to 'offset-path' which it references.
* Use a more precise description for the <<length-percentage>> value. TODO: The word "anchor point" should later be hyperlinked to a <dfn> instance under 'offset-anchor'.
* Use em-dash instead of hyphen, per English typographic rules.
* Use Bikeshed cross-reference syntax.
@tabatkins tabatkins added the ready label Nov 5, 2016
@fantasai fantasai changed the title Editorial improvements to offset-distance [motion] Editorial improvements to offset-distance Nov 5, 2016
@jihyerish jihyerish merged commit 89a316b into master Nov 9, 2016
@tabatkins tabatkins removed the ready label Nov 9, 2016
@plinss plinss deleted the fantasai-editorial-offset-distance branch August 18, 2017 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fantasai @tabatkins @jihyerish