Skip to content

Conversation

@beiz23
Copy link
Contributor

@beiz23 beiz23 commented Nov 29, 2017

  • Add a link to docs/general/device-pixel-ratio.md in docs/SUMMARY.md, since users could not reach the document on device-pixel-ratio.
  • fix small typos and unresolved anchors.

Copy link
Member

@simonbrunel simonbrunel left a comment

Choose a reason for hiding this comment

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

Looks really good, I would just rename the "Ratio" section to "Pixel Ratio"

docs/SUMMARY.md Outdated
* [Usage](getting-started/usage.md)
* [General](general/README.md)
* [Responsive](general/responsive.md)
* [Ratio](general/device-pixel-ratio.md)
Copy link
Member

Choose a reason for hiding this comment

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

I would call this section "Pixel Ratio" since there is also "Aspect Ratio" detailed in the Responsive section

Copy link
Member

@etimberg etimberg left a comment

Choose a reason for hiding this comment

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

I agree with @simonbrunel

Respond to changes request
@beiz23
Copy link
Contributor Author

beiz23 commented Nov 29, 2017

I renamed the "Ratio" section to "Pixel Ratio". Thank you for the quick reviews. 😄

@simonbrunel simonbrunel merged commit 9a7182b into chartjs:master Nov 30, 2017
@simonbrunel
Copy link
Member

Thanks @beiz23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants