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

review_activate script: Action name must be retrieved before performing the action #2982

Closed
nvaccessAuto opened this issue Feb 12, 2013 · 4 comments
Labels
Milestone

Comments

@nvaccessAuto
Copy link

@nvaccessAuto nvaccessAuto commented Feb 12, 2013

Reported by jteh on 2013-02-12 05:10
Str:

  1. Focus on a collapsed !SysTreeView32 item; e.g. in msinfo32.
  2. Press NVDA+numpadEnter to activate (expand) it.
  • Expected: NVDA should report "expand" as the action name.
  • Actual: NVDA reports "collapse" as the action name.

This occurs because the script retrieves the action name after performing the action instead of before. Once the action has been performed, it might change.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Feb 12, 2013

Comment 1 by jteh on 2013-02-12 05:10
Changes:
Changed title from "review_activate script: Action name must be fetched before performing th eaction" to "review_activate script: Action name must be retrieved before performing the action"

Loading

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Feb 12, 2013

Comment 2 by ateu on 2013-02-12 10:11
i think this ticket is associated with #2805

Loading

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Feb 12, 2013

Comment 3 by jteh on 2013-02-12 10:36
It's the same command, but in this case, you hear, for example, "collapse, expanded" instead of "expand, expanded".

Loading

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Mar 18, 2013

Comment 4 by mdcurran on 2013-03-18 05:58
Fixed in 07b911b.
Changes:
State: closed

Loading

@nvaccessAuto nvaccessAuto added this to the 2013.2 milestone Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant