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

resolves #660 use title case for page titles #661

Merged

Conversation

sturtison
Copy link
Contributor

Keep the same "Title Case Format for Page Titles" format as the core Asciidoctor documentation.

@sturtison
Copy link
Contributor Author

There are files in the samples folder that I didn't touch - not sure if they are used for testing.

@sturtison sturtison marked this pull request as ready for review October 21, 2023 12:42
@sturtison sturtison changed the title resolves #669 use title case for page titles resolves #660 use title case for page titles Oct 21, 2023
@sturtison
Copy link
Contributor Author

A check of the non-page section headings revealed some inconsistencies, so I fix those also.

@sturtison sturtison marked this pull request as draft October 21, 2023 13:59
* Use sentence case for non-page titles (asciidoctor#660)
* Keep the same format as the core Asciidoctor documentation.
@sturtison sturtison force-pushed the issue-660-upd-page-titles-to-title-case branch from fc1f7d6 to e75d264 Compare October 21, 2023 14:41
@sturtison sturtison marked this pull request as ready for review October 21, 2023 14:41
@ggrossetie ggrossetie merged commit 23630af into asciidoctor:main Oct 21, 2023
4 checks passed
@sturtison sturtison deleted the issue-660-upd-page-titles-to-title-case branch October 22, 2023 02:21
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