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

Clear the filter text when switching element types in Element List #5516

Closed
wants to merge 2 commits into from

Conversation

dkager
Copy link
Collaborator

@dkager dkager commented Nov 19, 2015

Fixes #5511.

@jcsteh
Copy link
Contributor

jcsteh commented Nov 25, 2015

Thanks! Unfortunately, this won't work. The filter method has a newElementType argument (defaulting to False) and it must be True when initialising for a new element type. Otherwise, the correct position in the tree is lost.

STR:

  1. Go to www.nvaccess.org.
  2. Press control+home.
  3. Press 2 to move to the "Home of the free NVDA screen reader" heading.
  4. Press NVDA+f7.
  5. Press alt+h.
  6. Press tab.
    • Expected: You should be on the "Home of the free NVDA screen reader" heading.
    • Actual: You're on the "NV Access" heading.

@dkager
Copy link
Collaborator Author

dkager commented Nov 25, 2015

Tis what you get for not reading docs. Fixed.

@dkager
Copy link
Collaborator Author

dkager commented Nov 26, 2015

Hmm, one more optimization could be to not perform the whole filtering algorithm, i.e. transforming all element names into lowercase, if the filter text is empty. Should give a slight performance boost unless Python is already clever enough to always mark "" in blah as true without evaluating blah.

@jcsteh
Copy link
Contributor

jcsteh commented Nov 26, 2015 via email

@jcsteh jcsteh closed this in 2df1a81 Dec 16, 2015
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

2 participants