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.

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

clearer headings in readme #88

Closed
wants to merge 1 commit into from
Closed

clearer headings in readme #88

wants to merge 1 commit into from

Conversation

wheresrhys
Copy link

@wheresrhys wheresrhys commented May 6, 2018

Due to github's styling of markdown headings & strong, the readme is a little unclear as to which parts are methods of store, and which are top level API methods. I've added a hr and changed the strongs to headings (which is arguably more semantically accurate) to try to improve the ease of scanning the docs.

There were also some mistakes (I think) in the heading hierarchy, so I have promoted some headings one or two levels

Due to github's styling of markdown headings & `strong`, the readme is a little unclear as to which parts are methods of `store`, and which are top level API methods. I've added a `hr` and changed the `strong`s  to `h6`s (which is arguably more semantically accurate) to try to improve the ease of scanning the docs.

There were also some mistakes (I think) in the heading hierarchy, so I have promoted headings one or two levels
@developit
Copy link
Owner

Hi @wheresrhys - I agree with your changes here, but unfortunately the "Usage" section of the README is actually generated by documentation.js, so it can't be modified directly.

@wheresrhys
Copy link
Author

Ah ok. I'll raise it over there. Another day, another yak to shave. Now, where did I put that extra large razor?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants