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

Add a release blurb and fixup changes for 2021.2 #12700

Merged
merged 7 commits into from
Aug 7, 2021
Merged

Conversation

seanbudd
Copy link
Member

@seanbudd seanbudd commented Jul 30, 2021

Description

Adds a release blurb and proposes some changes to the changelog and userGuide.

Testing strategy

I performed a review of git diff release-2021.1 userDocsChanges2021.2 user_docs/en/*.t2t.

The build artifact for changes.html and userGuide.html should be checked before merging.

Code Review Checklist:

  • Pull Request description is up to date.
  • Unit tests.
  • System (end to end) tests.
  • Manual testing.
  • User Documentation.
  • Change log entry.
  • Context sensitive help for GUI changes.
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers

@seanbudd seanbudd added this to the 2021.2 milestone Jul 30, 2021
@seanbudd seanbudd requested a review from Qchristensen July 30, 2021 01:02
@seanbudd seanbudd requested review from a team as code owners July 30, 2021 01:02
@seanbudd seanbudd requested review from michaelDCurran and feerrenrut and removed request for a team July 30, 2021 01:02

== Important notes ==
- Screen Curtain on previous releases of NVDA is no longer compatible with versions of Windows newer than Windows 10 21H2 (10.0.19044).
Screen Curtain is no longer compatible when using versions of NVDA older than 2021.2 with Windows newer than Windows 10 21H2 (10.0.19044).
Copy link
Contributor

Choose a reason for hiding this comment

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

newer than

Shouldn't this be "when using x or later"?

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually yes, that is the case as far as I'm aware. Before I fix this (across the rest of the documentation) I'm going to check this via the release preview channel for WIN1021H2. I have confirmed the previous release of Windows works, Windows 10 21H1.

Copy link
Member Author

Choose a reason for hiding this comment

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

The change of behaviour occurred in an insiders build, #12491 reports this change of behaviour for a build sometime after the WIN10 21H2 build number. I am yet to be able to test WIN10 21H2 preview (haven't been offered the update). It would be good if someone can confirm whether or not screen curtain works with the WIN10 21H2 preview, but for now I will move forward to say "when using it or later" due to being uncertain.

user_docs/en/changes.t2t Outdated Show resolved Hide resolved
@seanbudd seanbudd requested a review from feerrenrut August 3, 2021 07:25
michaelDCurran
michaelDCurran previously approved these changes Aug 5, 2021
Copy link
Member

@michaelDCurran michaelDCurran left a comment

Choose a reason for hiding this comment

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

This looks good to me.

user_docs/en/changes.t2t Outdated Show resolved Hide resolved
feerrenrut
feerrenrut previously approved these changes Aug 5, 2021
Copy link
Contributor

@feerrenrut feerrenrut left a comment

Choose a reason for hiding this comment

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

Overall I think this is fine.

Just a couple of ideas, trying to make the language for the screen curtain edge cases as simple as possible.

user_docs/en/changes.t2t Outdated Show resolved Hide resolved
@seanbudd seanbudd dismissed stale reviews from feerrenrut and michaelDCurran via 00fa3ee August 6, 2021 00:11
@seanbudd
Copy link
Member Author

seanbudd commented Aug 6, 2021

@feerrenrut - I've rephrased this message across both files now. Let me know what you think.

@seanbudd seanbudd requested a review from feerrenrut August 6, 2021 00:13
@seanbudd seanbudd merged commit 7028d2e into beta Aug 7, 2021
@seanbudd seanbudd deleted the userDocsChanges2021.2 branch August 7, 2021 02:44
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