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

User guide and what's new for 2017.3: spelling fixes, grammar changes, making user guide friendlier, edits to what's new. re #7371 #7373

Merged
merged 6 commits into from Jul 11, 2017

Conversation

josephsl
Copy link
Collaborator

@josephsl josephsl commented Jul 11, 2017

Hi,

Resolves the following:

  • Fixes Spelling Errors in User guide #7371: numerous issues in user guide such as spelling inconsistencies, making user guide friendlier, term changes, capitalizations.
  • What's new edits.

Thanks.

…nvaccess#7371.

Changes include:
* Added 'Welcome to NVDA!' at the beginning of the user guide to make it even friendlier.
* Realising: realizing.
* Licence: license.
* Organised: organized.
* Combobox: combo box.
* Capitalized features list and language names.
Copy link
Contributor

@jcsteh jcsteh left a comment

Choose a reason for hiding this comment

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

Thanks! A few comments:

  1. You mention What's New edits, but I don't see any. Did you forget to push them?
  2. Would you mind fixing the misspelling of "Tells" as "Tels" at around line 2343?
  3. See two other review comments below.

@@ -8,17 +8,18 @@ NVDA NVDA_VERSION User Guide
%%toc

+ Introduction +
Welcome to NVDA!
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a blank line after this so it's in a paragraph by itself.

@@ -247,7 +248,7 @@ When the menu comes up, You can use the arrow keys to navigate the menu, and the
| Toggle Input Help Mode | NVDA+1 | NVDA+1 | none | Pressing any key in this mode will report the key, and the description of any NVDA command associated with it |
| Quit NVDA | NVDA+q | NVDA+q | none | Exits NVDA |
| Pass next key through | NVDA+f2 | NVDA+f2 | none | Tells NVDA to pass the next key press straight through to the active application, even if it is normally treated as an NVDA key command |
| Toggle application sleep mode on and off | NVDA+shift+s | NVDA+shift+z | none | sleep mode disables all NVDA commands and speech/braille output for the current application. This is most useful in applications that provide their own speech or screen reading features. Press this command again to disable self voicing mode. |
| Toggle application sleep mode on and off | NVDA+shift+s | NVDA+shift+z | none | sleep mode disables all NVDA commands and speech/braille output for the current application. This is most useful in applications that provide their own speech or screen reading features. Press this command again to disable self-voicing mode. |
Copy link
Contributor

Choose a reason for hiding this comment

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

I think "self-voicing mode" here should be "sleep mode", as we're calling it two different things otherwise.

@josephsl
Copy link
Collaborator Author

josephsl commented Jul 11, 2017 via email

… > sleep mode. re nvaccess#7371.

Reviewed by Jamie Teh (NV Access):
* Tels -> tells
* Self-voicing > sleep mode
* Welcome text is on its own paragraph.
@@ -8,9 +8,9 @@
== New Features ==
- A Braille setting has been added to "show messages indefinitely". (#6669)
- In Microsoft Outlook message lists, report if a message is flagged. (#6374)
- In Microsoft Powerpoint, the exact type of a shape is now reported when editing a slide (Examples include: triangle, circle, video, arrow), rather than just 'shape'. (#7111)
- In Microsoft PowerPoint, the exact type of a shape is now reported when editing a slide (Examples include: triangle, circle, video, arrow), rather than just 'shape'. (#7111)
Copy link
Contributor

Choose a reason for hiding this comment

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

This "examples include" thing is a bit weird linguistically. Maybe we could change it to this:

- In Microsoft PowerPoint, the exact type of a shape is now reported when editing a slide (such as triangle, circle, video or arrow), rather than just "shape". (#7111)

- Mathematical content (provided as MathML) is now supported in Google Chrome. (#7184)
- NVDA can now speak using the new Windows OneCore voices (also known as mobile voices) included in Windows 10. You access these by selecting Windows OneCore voices in NVDA's Synthesizer dialog. (#6159)
- NVDA can now speak using the new Windows OneCore voices (also known as SAPI5 Mobile voices) included in Windows 10. You access these by selecting Windows OneCore voices in NVDA's Synthesizer dialog. (#6159)
Copy link
Contributor

Choose a reason for hiding this comment

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

No one calls these SAPI5 Mobile apart from an old NVDA driver. Also, the voices were previously named "Mobile"; e.g. "Microsoft Zira Mobile". I don't think SAPI5 should be mentioned here.

@@ -37,7 +37,7 @@
- "form": "frm"
- "region": "rgn"
- Example: "navigation landmark" becomes "lmk nav"
- Espeak-ng has been updated to release 1.49.1 (#7280).
- Updated Espeak-ng to 1.49.1. (#7280).
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. This should be written "eSpeak NG" as is the official title of the project.
  2. Please remove trailing full stop at end of line.

@josephsl
Copy link
Collaborator Author

josephsl commented Jul 11, 2017 via email

@derekriemer
Copy link
Collaborator

  • If a dialog contains a progress bar, the dialog text is now updated when the progress bar changes. This means, for example, that the remaining time can now be read in NVDA's "Downloading Update" dialog. (Braille doesn't refresh when downloading update #6862)
    should this be for braille?

@josephsl
Copy link
Collaborator Author

josephsl commented Jul 11, 2017 via email

@jcsteh
Copy link
Contributor

jcsteh commented Jul 11, 2017

@derekriemer commented on 11 Jul 2017, 13:34 GMT+10:

  • If a dialog contains a progress bar, the dialog text is now updated when the progress bar changes.

My bad; this should be "the dialog text is updated in braille when...".

@jcsteh
Copy link
Contributor

jcsteh commented Jul 11, 2017

@josephsl commented on 11 Jul 2017, 13:11 GMT+10:

Hi, done. For item 3, Reef’s work. Thanks.

I know. I'm just more linguistically pedantic. There's a reason I normally review the What's New thoroughly just before release freeze. :)

@jcsteh jcsteh merged commit caeb192 into nvaccess:master Jul 11, 2017
@nvaccessAuto nvaccessAuto added this to the 2017.3 milestone Jul 11, 2017
@josephsl josephsl deleted the 17.3docs branch July 19, 2020 22:17
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.

None yet

4 participants