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

Separate use-cases, quick start pages: (asciidoctor#652) (asciidoctor#654) #659

Merged

Conversation

sturtison
Copy link
Contributor

@sturtison sturtison commented Oct 15, 2023

Separate use-cases and quick start pages: #652 #654

Create separate pages from existing content into:

  • use-cases.adoc
  • quickstart.adoc
  • diagrams-extension-quickstart.adoc

Update nav.adoc.

Resulting navigation:
image

@sturtison sturtison force-pushed the issue-654-reorg-use-cases-quickstart branch from b27b3e2 to 8d23f85 Compare October 15, 2023 05:53
…tor#654)

Create separate pages from existing content into:

* use-cases.adoc
* quickstart.adoc
* diagrams-extension-quickstart.adoc

Update nav.adoc.
@sturtison sturtison force-pushed the issue-654-reorg-use-cases-quickstart branch from 8d23f85 to 7c4f821 Compare October 15, 2023 06:02
@sturtison sturtison changed the title Update changelog.adoc Separate use-cases, quick start pages: (asciidoctor#652) (asciidoctor#654) Oct 15, 2023
@sturtison sturtison marked this pull request as ready for review October 15, 2023 06:05
@sturtison
Copy link
Contributor Author

Content moved, links and nav adjusted.
Some content unchanged but rearranged. e.g. move "you will learn" to the top of the page.

@sturtison
Copy link
Contributor Author

sturtison commented Oct 15, 2023

After:
image

@ggrossetie
Copy link
Member

Thanks!

@ggrossetie ggrossetie merged commit fb26526 into asciidoctor:main Oct 20, 2023
4 checks passed
@mojavelinux
Copy link
Member

I like the idea of introducing more pages so that each page is more focused and digestible.

I would like to point out that the page titles now have inconsistent capitalization. Could we agree on a style and stick to it. See https://docs.asciidoctor.org/browser-extension/diagrams-extension-quickstart/ Notice "Extension Options" verse "Diagrams extension quickstart". In the core documentation, we tend to use title capitalization for page titles. See https://docs.asciidoctor.org/asciidoctor/latest/

@sturtison sturtison deleted the issue-654-reorg-use-cases-quickstart branch October 21, 2023 12:15
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.

3 participants