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

add warning note that NLS eReader Zoomax is not supported #16628

Merged
merged 2 commits into from
May 30, 2024

Conversation

seanbudd
Copy link
Member

Link to issue number:

Related to #15863

Summary of the issue:

The NLS eReader Zoomax is not supported by NVDA currently, despite a message in the user guide suggesting the contrary.
See #15863 for discussion on adding support for the eReader

Description of user facing changes

add warning note that NLS eReader Zoomax is not supported

@seanbudd seanbudd requested a review from a team as a code owner May 29, 2024 05:39
@seanbudd seanbudd requested a review from Qchristensen May 29, 2024 05:39
Copy link
Member

@Qchristensen Qchristensen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe the Zoomax device does work with NVDA, with a driver supplied by NLS, so we should perhaps say that. This line may give the impression that it does not work with NVDA at all.

user_docs/en/userGuide.md Outdated Show resolved Hide resolved
@seanbudd seanbudd merged commit 1957058 into master May 30, 2024
1 of 2 checks passed
@seanbudd seanbudd deleted the noteNLSZoomax branch May 30, 2024 01:06
@XLTechie
Copy link
Collaborator

XLTechie commented May 30, 2024 via email

@seanbudd
Copy link
Member Author

@XLTechie - as discussed with Quentin, the context of this text is:

The following extra devices are also supported (and do not require any special drivers to be installed):

So the goal here is just to clarify that the Zoomax is an exception to the need for external software. Perhaps both this part and the zoomax exception needs clarification that drivers/add-ons are needed.

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.

3 participants