-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fixed capitalization of subsections in chapter 7. Response #981
Conversation
✅ Deploy Preview for graphql-spec-draft ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Notes from the Dec APAC WG: thanks for spotting this inconsistency. We need to decide on a style guide for the spec and then uniformly apply it across the entire document for all |
I've used:
To pull out all the titles of this format. They roughly sort into: Title caseSeems like deliberate title case.
Title case via keywordsSeems like title case but could just be that capitalised keywords such as
Sentence caseUnambiguously sentence case.
Sentence case except keywordsCapitalisation of keywords (
IndeterminateSingle words don't sway one way or the other.
One more thing...There's inconsistency on whether we should capitalise after a hyphen; e.g. we have |
After a deeper analysis, I have proposed this: #1003 |
Perfect! |
Since the new style guide sets the Title case for headings, this PR fixes the remaining non-conforming headers to Title case, so it is ready to merge. |
Changed capitalization of subsections to first-cap style like in the rest of the doc