UI/doc tweaks for "Announce formatting changes after the cursor" #6336

Merged
merged 2 commits into from Oct 18, 2016

Conversation

Projects
None yet
3 participants
@jcsteh
Contributor

jcsteh commented Sep 6, 2016

In NVDA's Document Formatting settings, "Announce formatting changes after the cursor" has been renamed to "Report formatting changes after the cursor", as it affects braille as well as speech.
Also, the User Guide previously gave Microsoft Word as an example of an application where this setting needs to be enabled. However, this setting is never checked for Word. Change this example to Wordpad instead.

In NVDA's Document Formatting settings, "Announce formatting changes …
…after the cursor" has been renamed to "Report formatting changes after the cursor", as it affects braille as well as speech.

Also, the User Guide previously gave Microsoft Word as an example of an application where this setting needs to be enabled. However, this setting is never checked for Word. Change this example to Wordpad instead.
@jcsteh

This comment has been minimized.

Show comment
Hide comment
@jcsteh

jcsteh Sep 6, 2016

Contributor

@feerrenrut, mind taking a look?

Contributor

jcsteh commented Sep 6, 2016

@feerrenrut, mind taking a look?

@feerrenrut

This comment has been minimized.

Show comment
Hide comment
@feerrenrut

feerrenrut Sep 6, 2016

Contributor

@jcsteh Looks good to me

Contributor

feerrenrut commented Sep 6, 2016

@jcsteh Looks good to me

jcsteh added a commit that referenced this pull request Sep 6, 2016

@nvaccessAuto nvaccessAuto assigned jcsteh and unassigned feerrenrut Sep 6, 2016

@jcsteh

This comment has been minimized.

Show comment
Hide comment
@jcsteh

jcsteh Sep 29, 2016

Contributor

This is incorporated into #6402. There is one tiny User Guide change that isn't included yet, however. CC @feerrenrut.

Contributor

jcsteh commented Sep 29, 2016

This is incorporated into #6402. There is one tiny User Guide change that isn't included yet, however. CC @feerrenrut.

@feerrenrut

This comment has been minimized.

Show comment
Hide comment
@feerrenrut

feerrenrut Sep 30, 2016

Contributor

@jcsteh I didn't back out this pull request, the conflicts were minimal. If all changes required are included in this PR then we are good to go. This PR will still need to be merged to master once incubated.

Contributor

feerrenrut commented Sep 30, 2016

@jcsteh I didn't back out this pull request, the conflicts were minimal. If all changes required are included in this PR then we are good to go. This PR will still need to be merged to master once incubated.

@jcsteh jcsteh merged commit 27bc4e4 into master Oct 18, 2016

@nvaccessAuto nvaccessAuto added this to the 2016.4 milestone Oct 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment