Skip to content

fix up documentation changes for 2024.2#16423

Merged
seanbudd merged 8 commits into
betafrom
fixUpReleaseBlurb
May 10, 2024
Merged

fix up documentation changes for 2024.2#16423
seanbudd merged 8 commits into
betafrom
fixUpReleaseBlurb

Conversation

@seanbudd

Copy link
Copy Markdown
Member

No description provided.

@seanbudd seanbudd added this to the 2024.2 milestone Apr 18, 2024
@seanbudd seanbudd changed the base branch from master to beta April 18, 2024 05:56
@CyrilleB79

Copy link
Copy Markdown
Contributor

In the blurb, I can read:

When active, the braille display shows exactly what NVDA prepares to speak.

IMO, "prepare to speak" is not clear for end-users.

We may just write:
"When active, the braille display shows exactly what NVDA speaks."

For the blurb, it's not important to take into account corner cases such as the "speech off" case.

@CyrilleB79

Copy link
Copy Markdown
Contributor

By the way, also for the subsequent change log item, we may remove "prepare to".

The User Guide is more precise and indicates:

When display speech output is selected, the braille display will show what NVDA speaks, or would have spoken if speech mode was set to "talk"

But this corner case does not need to be presented in the change log IMO.

@CyrilleB79

Copy link
Copy Markdown
Contributor

By the way, if this PR gathers all minor changes needed in the documentation, in the User Guide, could you add a period at the end of the following sentence?

When display speech output is selected, the braille display will show what NVDA speaks, or would have spoken if speech mode was set to "talk"

@seanbudd seanbudd marked this pull request as ready for review April 22, 2024 06:21
@seanbudd seanbudd requested review from a team as code owners April 22, 2024 06:21
@seanbudd seanbudd requested review from Qchristensen and michaelDCurran and removed request for a team April 22, 2024 06:21
@seanbudd

Copy link
Copy Markdown
Member Author

thanks @CyrilleB79 for the catches

@seanbudd seanbudd changed the title fix up release blurb fix up documentation changes for 2024.2 Apr 22, 2024
Qchristensen
Qchristensen previously approved these changes Apr 22, 2024

@Qchristensen Qchristensen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reads well

michaelDCurran
michaelDCurran previously approved these changes Apr 22, 2024
@seanbudd seanbudd added conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. queued for merge labels Apr 23, 2024
@CyrilleB79

Copy link
Copy Markdown
Contributor

@seanbudd while at it, could you also fix here the table formatting issues described by @k-kolev1985 in this message on the translators mailing list?
Thanks.

@k-kolev1985

Copy link
Copy Markdown
Contributor

Also, I've noticed that some heading titles (marked with ====) are missing a space between the text and the ==== sequence. I'm not sure if this causes a formatting breakage or not, but I'm reporting it just in case. Unfortunately, I did not wrote down on which lines in the documentations are those missing spaces.

@seanbudd seanbudd dismissed stale reviews from michaelDCurran and Qchristensen via c32c922 April 30, 2024 06:32
@seanbudd

Copy link
Copy Markdown
Member Author

@CyrilleB79 - I've fixed this up, thanks @k-kolev1985 for reporting this

@k-kolev1985 - I don't think this is causes any formatting issues, and also should be fixed when we migrate to markdown next week.

Comment thread user_docs/en/changes.t2t Outdated
Comment thread user_docs/en/changes.t2t Outdated
Comment thread user_docs/en/changes.t2t Outdated
Co-authored-by: Luke Davis <8139760+XLTechie@users.noreply.github.com>

@CyrilleB79 CyrilleB79 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the User Guide:
"in a larger step" -> "in larger steps"

Please check all occurrences.

Comment thread user_docs/en/userGuide.t2t Outdated
Comment thread user_docs/en/userGuide.t2t Outdated
seanbudd and others added 2 commits May 1, 2024 11:04
@seanbudd seanbudd merged commit 3d16145 into beta May 10, 2024
@seanbudd seanbudd deleted the fixUpReleaseBlurb branch May 10, 2024 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. queued for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants