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

README: Table of axis definitions #117

Closed
2 tasks done
davelab6 opened this issue Nov 18, 2019 · 2 comments
Closed
2 tasks done

README: Table of axis definitions #117

davelab6 opened this issue Nov 18, 2019 · 2 comments

Comments

@davelab6
Copy link
Contributor

davelab6 commented Nov 18, 2019 •
edited

The README showing off the axes is great! Just a couple of small requests

  • Please add a table to the README.md with the axis label, tag, and min, default, max range definitions (similar to the table at https://github.com/arrowtype/recursive/#variable-axes)

  • I suggest to use an easing method on the GIFs so that the ends of the axis are given enough time; since the axes are 'curved' with avar data, the 'linear' GIF animation doesn't illustrate what is happening well (especially the ungoofing of opsz-min :)
@davelab6 davelab6 mentioned this issue Nov 18, 2019
Closed
@sponcey
Copy link
Collaborator

sponcey commented Nov 18, 2019 •
edited

Great feedback, will incorporate this.

@davelab6
Copy link
Contributor Author

Closing as both are done :) Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davelab6 @sponcey