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

Revert selection message being at the end for list cases #9894

Closed
derekriemer opened this issue Jul 7, 2019 · 12 comments
Closed

Revert selection message being at the end for list cases #9894

derekriemer opened this issue Jul 7, 2019 · 12 comments

Comments

@derekriemer
Copy link
Collaborator

Steps to reproduce:

  1. open a windows explorer window, select the files list, or open any multi-select list.
  2. press control+space on a file.
  3. move down or up with control+arrows.

Actual behavior:

NVDA says "blah blah blah" before selected or unselected.

Expected behavior:

As before, NVDA should say selected or unselected before the message, but only when selection changes.

NVDA version:

Version: alpha-17928,9dfbd2ee

Regression caused by #9028

@derekriemer
Copy link
Collaborator Author

@LeonarddeR @feerrenrut

@josephsl
Copy link
Collaborator

josephsl commented Jul 7, 2019 via email

@DrSooom
Copy link

DrSooom commented Jul 8, 2019

I love checkboxes. Well, I guess all of you already knew that. 😉

@derekriemer
Copy link
Collaborator Author

No, I think we should just remove the new behavior in this specific case. it's a special case. We don't need NVDA being littered with hundreds of one-off decision checkboxes.

@Brian1Gaff
Copy link

Brian1Gaff commented Jul 8, 2019 via email

@Brian1Gaff
Copy link

Brian1Gaff commented Jul 8, 2019 via email

@lukaszgo1
Copy link
Contributor

I am testing with 2019.1.1 and selected/nos selected is reported at the end in these cases. @derekriemer Are your certain that it is a regression?

@LeonarddeR
Copy link
Collaborator

@lukaszgo1 Are you testing with the English version of NVDA? It could be that the translators of your particular language decided to swap the order.

I agree with @derekriemer. the selected state should be announced before the list item in these cases.

@lukaszgo1
Copy link
Contributor

Yes, I've tested with the 2019.1.1 in English, and not selected is announced at the end, selected however isn't announced at all when navigating with CTRL+arrows.

@DrSooom
Copy link

DrSooom commented Jul 9, 2019

Yes, I've tested with the 2019.1.1 in English, and not selected is announced at the end, selected however isn't announced at all when navigating with CTRL+arrows.

Exactly the same behavior on my machines with NVDA 2015.3/2018.1 (I'm waiting for 2019.2) on Win7x64 – both German GUI (incl. NVDA).

It could be that the translators of your particular language decided to swap the order.

I suppose it wouldn't be bad to talk about this in a separate issue or in the translators' mailing list.

@XLTechie
Copy link
Collaborator

XLTechie commented Jul 10, 2019 via email

@feerrenrut
Copy link
Contributor

I don't believe there is a regression here so I'm going to close this issue. If you get different results on some system please add a comment.

For each preference (selected first vs last) there are good justifications. Collecting these user stories may help us to innovate in this area, see #6757 (comment) for an example of similar user stories.

Sometimes this is just a matter of preference, NV Access has goals to improve the configurability of the NVDA speech user experience and even suggested it as a project idea for Google Summer of Code. However, creating a check box for every variation will make configuration for users difficult and maintenance for developers difficult.

Test method

  1. In the keyboard settings panel (NVDA + control + k) enable:
    • Speak command keys
  2. Open speech viewer
  3. Open explorer, navigate to a folder with at least 5 items
  4. Focus the second item in the the file list
  5. Press NVDA + F12 to speak the time, marking the start of speech output
  6. Press NVDA + TAB to ensure the name of the currently focused file is spoken
  7. Press shift + down arrow once to select two consecutive items
  8. Press control + down arrow twice to skip an item
  9. Press control + space to select the non consecutive item
  10. Press control + up arrow four times to focus the first list item
  11. Alt + tab to the speech viewer, select text from the alt+tab command, back to the time marker, copy this as your test output.

Results

For 2019.1, latest beta build, and 2017.3 the output was all identical ( except for the time). See detailed output below.

2019.1:

11:30 AM
NVDA+tab
test file B.txt  list item  focused  selected  2 of 6
shift+down arrow
test file C.txt  3 of 6
ctrl+down arrow
test file D.txt  not selected  4 of 6
ctrl+down arrow
test file E.txt  not selected  5 of 6
ctrl+space
selected
ctrl+up arrow
test file D.txt  not selected  4 of 6
ctrl+up arrow
test file C.txt  3 of 6
ctrl+up arrow
test file B.txt  2 of 6
ctrl+up arrow
test file A.txt  not selected  1 of 6
alt+tab

Latest Beta (beta-18022,dc5c0b8c)

11:33 AM
NVDA+tab
test file B.txt  list item  focused  selected  2 of 6
shift+down arrow
test file C.txt  3 of 6
ctrl+down arrow
test file D.txt  not selected  4 of 6
ctrl+down arrow
test file E.txt  not selected  5 of 6
ctrl+space
selected
ctrl+up arrow
test file D.txt  not selected  4 of 6
ctrl+up arrow
test file C.txt  3 of 6
ctrl+up arrow
test file B.txt  2 of 6
ctrl+up arrow
test file A.txt  not selected  1 of 6
alt+tab

2017.3

11:37 AM
NVDA+tab
test file B.txt  list item  focused  selected  2 of 6
shift+down arrow
test file C.txt  3 of 6
ctrl+down arrow
test file D.txt  not selected  4 of 6
ctrl+down arrow
test file E.txt  not selected  5 of 6
ctrl+space
selected
ctrl+up arrow
test file D.txt  not selected  4 of 6
ctrl+up arrow
test file C.txt  3 of 6
ctrl+up arrow
test file B.txt  2 of 6
ctrl+up arrow
test file A.txt  not selected  1 of 6
alt+tab

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

No branches or pull requests

8 participants