Skip to content

added short description + links to MkDocs docu#6267

Merged
squidfunk merged 2 commits intosquidfunk:masterfrom
alexvoss:docu_gh-deploy
Nov 2, 2023
Merged

added short description + links to MkDocs docu#6267
squidfunk merged 2 commits intosquidfunk:masterfrom
alexvoss:docu_gh-deploy

Conversation

@alexvoss
Copy link
Copy Markdown
Collaborator

@alexvoss alexvoss commented Nov 1, 2023

I thought the description of how to deploy from your local machine to GH Pages was a bit brief.

@alexvoss alexvoss requested a review from squidfunk November 1, 2023 10:07
Copy link
Copy Markdown
Owner

@squidfunk squidfunk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good addition! Minor changes.

Comment thread docs/publishing-your-site.md Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"See this section" is not a good link name from an A11y point of view. We try to minimize those, because when you step with tab through the docs, your screenreander will not provide enough context. We definitely have some parts where we also do it like that, but when updating our documentation we should always try to give links discernible names, so that they can be understood on their own.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hope the wording is better now. Also tried to avoid the repetition of "MkDocs documentation".

@squidfunk squidfunk merged commit 9a7a185 into squidfunk:master Nov 2, 2023
@alexvoss alexvoss deleted the docu_gh-deploy branch December 1, 2023 19:12
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