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

UX: NVDA+F3, NVDA+Shift+F3, Find Error #7422

Open
dan1982code opened this issue Jul 19, 2017 · 17 comments
Open

UX: NVDA+F3, NVDA+Shift+F3, Find Error #7422

dan1982code opened this issue Jul 19, 2017 · 17 comments
Labels

Comments

@dan1982code
Copy link

When in browse mode and you hit something like q, and there is no next block quote, we get a spoken error but no alert box.

But when we do NVDA+F3 to search for text, and the text isn't found, we get an alert box with a "find error" in it. This requires the additional keystroke of hitting escape to clear this dialog.

Could the "find error" be spoken just as is the "no next block quote" message; i.e. with no alert box shown?

I know... it is just a single escape keystroke to clear the "find error" dialog, but often I will do NVDA+F3 successively on a bunch of PDF pages to find what I want. The escape keypress and Windows alert sound each time could be avoided if we changed to not use an alert.

I also wonder whether this is a user interface consistency issue, or if a "not found" (q) is different from an "error". I also wonder whether it's really a "find error". If it is a "find error" then it should be a "block quote error", which doesn't make much sense.

@jcsteh
Copy link
Contributor

jcsteh commented Jul 19, 2017 via email

@ruifontes
Copy link
Contributor

ruifontes commented Jul 19, 2017 via email

@dan1982code
Copy link
Author

Thanks! I agree in that we're trading one inconsistency for another. That said, there's no way we can do NVDA+ctrl+F without a dialog box. So the rule, consistently applied, could be: don't use a dialog/alert unless required. With that rule, NVDA+F3 could be consistent in not bringing up an alert when not found.

@dan1982code
Copy link
Author

I don't quite think that this fixes 7415. You still couldn't use NVDA+CTRL+F to find something new. You could use NVDA+F3 to search for the next occurrence of the last find, though, and a "find error" wouldn't ruin it.

@jcsteh
Copy link
Contributor

jcsteh commented Jul 19, 2017 via email

@dan1982code
Copy link
Author

Misread, sorry :( Cool, though! We could fix that and remove the (arguably) extraneous "find error" alert.

@lukaszgo1
Copy link
Contributor

@michaelDCurran @feerrenrut Any thoughts on this? For me it seems to be a great addition, and if there are no objections I am going to implement this. The only thing which worries me a little are braille only users for whom the flash message maybe not enough to realize what has happened. @LeonarddeR What is your opinion of this as a braille user?

@zstanecic
Copy link
Contributor

zstanecic commented Jan 10, 2019 via email

@lukaszgo1
Copy link
Contributor

Limit definitely can be increased, however we have to ensure, that everything works nicely with the default config.

@Adriani90
Copy link
Collaborator

@lukaszgo1 are you still intending to raise a Pull request for this issue?

@lukaszgo1
Copy link
Contributor

I've implemented it, worked with my implementation for a few hours. and I am no longer convinced that it is a good solution. The main problem is with the fact that before message about text not found the part of the document title is spoken. It works the same when the text is found, but now when the dialog pops-up the error sound is a good indication that we should dismiss it and search again, so there is no speed benefit in my view.

@Adriani90
Copy link
Collaborator

Actually, I was thinking about an alternative Approach. Instead of giving a warning by speech, there could be a very specific Sound indicating that no results have been found. A specific Sound like spelling error but for "no results found". In fact the Dialog would be obsolete because the only Action possible is to press "ok". But if #4637 will be implemented, the Dialog is needed anyway for the additional Action to search again from top. In conclusion, I think having this Dialog makes sense because Features can be added to it in the future.
Thus, I am closing this as won't fix. If anyone has other arguments, please comment on it and we can reopen the discussion.

@lukaszgo1
Copy link
Contributor

If #4637 would be implemented by @marlon-sousa the wrapping would be configurable, therefore some announcement about text not found would be still needed. I believe we should leave this open until #4637 is implemented and then com back to it.
Alternatively if someone could provide appropriate sound I can open a PR with @Adriani90 idea.

@Adriani90
Copy link
Collaborator

Reopening and adding blocked Label since the implementation depends on #4637.

@Adriani90
Copy link
Collaborator

One major issue with the current dialog approach is that the location of the virtual cursor is lost in complex documents such as large pdf documents when they are opened in a browser. I think it would be more convenient to have a notification like "no further results" instead of the dialog.
In case of large pdfs, NVDA has to find the previously focused text in the browse mode document after dismissing the dialog or pressing ok. In some large pdf documents NVDA freezes for more than 30 seconds until you can navigate in browse mode again.

@jcsteh
Copy link
Contributor

jcsteh commented Apr 22, 2024

If there are cases where NVDA takes a long time to restore its position in browse mode, that should be filed as a separate issue if it isn't already. That shouldn't really happen unless the document is doing something very strange like re-rendering the entire document. Also, that would have impacts far beyond the NVDA+f3 command. It would impact switching applications, opening any other dialog (even the browse mode find dialog itself), etc. Thus, that is somewhat separate from the request described here.

@Adriani90
Copy link
Collaborator

I creaded #16451.

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

No branches or pull requests

6 participants