Skip to content

Readme update #135

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Readme update #135

wants to merge 9 commits into from

Conversation

scolsen
Copy link

@scolsen scolsen commented Oct 22, 2018

Adds documentation on enabling folding using g:markdown_folding. Introduces some parallel grammar and fixes minor typos. Adds some section headers and styling for readability.

@anthony-geoghegan
Copy link
Contributor

I think these are all useful commits and it would be great to see this pull-request merged.

I’d suggest a few further changes:

  • wrap the long lines (those beginning with Note)
  • change the Fenced Codeblock Highlighting heading to Fenced code block highlighting – “code block” is usually written as two words in Markdown documentation sources (including this file).

Also, the top-level heading of the initial README uses the style of capitalising only the first word (and proper nouns such as “Markdown”) so it would make sense to be consistent in using this style for all headers.

@scolsen
Copy link
Author

scolsen commented Apr 16, 2020

@anthony-geoghegan thanks for the bump! Incorporated your suggestions.

@anthony-geoghegan
Copy link
Contributor

Nice one, @scolsen Let's hope it gets merged. I recently opened a pull-request, myself but I'm guessing Tim has more important things on his mind in these troubled times.

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

Successfully merging this pull request may close these issues.

2 participants