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

If reading a word and it contains only a symbol and white space, Read symbol irrespective of symbol level #5133

Closed
nvaccessAuto opened this issue Jun 2, 2015 · 4 comments · Fixed by #11167
Labels
bug component/speech p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority
Milestone

Comments

@nvaccessAuto
Copy link

Reported by sumandogra on 2015-06-02 09:09
The punctuation marks such as comma are not read with word reading commands until and unless the punctuation marks is set to all in the voce dialog box. It is suggested to make these punctuations marks read on using word reading and character reading command irrespective of the punctuation marks level, NVDA remains silent on reaching the punctuation Marks such as comma with word reading command if the punctuation marks is not set to all.

Steps to reproduce:

  1. Open a word document that has many punctuation marks such as commas.
  2. Read the document with word reading command.
  3. On reaching comma, NVDA remains silent.
@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2015-06-03 00:26
Doing this for all words could be undesirable for some. If the user has chosen to read a certain level of punctuation, it's reasonable to expect that this should always be the case except when reading by character. Doing otherwise could create inconsistency and confusion.

That said, we should probably make sure we speak all words as characters if they contain only a single character. We already do this in some cases, but not all.

Technical: We don't seem to speak single character words as characters if the single character is followed by a space. We should probably strip spaces before doing this test.

@jcsteh jcsteh changed the title suggestion: NVDA to read the punctuation marks with word reading command irrespective of the level of the punctuation marks. If reading a word and it contains only a symbol and white space, Read symbol irrespective of symbol level Aug 6, 2017
@jcsteh jcsteh added the p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority label Aug 6, 2017
@jcsteh
Copy link
Contributor

jcsteh commented Aug 6, 2017

P3 because there's an easy workaround (change the symbol level temporarily or move by character if you hear silence when moving by word).

Implementation note: this shouldn't be too difficult. In speech.py, in speakTextInfo around line 853, we check len(textWithFields[0])==1. This needs to be changed so that we strip white space from textWithFields[0] before checking the length. Note that we shouldn't strip if the text contains only white space, since that would cause, for example, single tab characters to be silenced when moving by word.

@Adriani90
Copy link
Collaborator

I suggest to close this issue because the user can specify in the symbols dialog on which symbol level NVDA should report a certain symbol. So you can change the reporting behavior for Komma to be reported when you have symbols set to another setting than "all". Note that it works for example properly for dot, percent or dollar sign if you have punctuation to "some". For komma it can be adjusted accordingly.
cc: @jcsteh

@Qchristensen
Copy link
Member

I'm not a fan of situations where you can move the focus and have NVDA report nothing, as is the case if you control+right arrow through a sentence with a comma (particularly in Word, for instance). I agree with the proposal to have NVDA report the punctuation where it is the only text to be spoken, even if the punctuation level wouldn't have otherwise had that symbol spoken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug component/speech p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority
Projects
None yet
4 participants