-
Notifications
You must be signed in to change notification settings - Fork 377
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.
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
Restructure README #221
Restructure README #221
Conversation
hmm, maybe we don't need the icons anymore… |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add the D3 logo at the top of the README (like we do on d3/d3) and then use a complete sentence to introduce the library. You can copy the first paragraph from https://d3js.org/d3-force.
OK, I've minimized it again (preview). Also, using the D3 logo from d3/d3 — would it be any better to have a local copy? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you could keep the emoji in the bulleted list to be a little more “fun”. I like the conciseness of the list here.
I like the emojis just to make it a little less dry. We can add them later. |
This proposal (see page):
For this particular module, the reference to beeswarm is obsolete. (Though I still like the old beeswarm, and am tempted to make a notebook for it 🤓).
Once we settle on a template, this will have to be replicated to all the modules covered by d3js.org