-
-
Notifications
You must be signed in to change notification settings - Fork 659
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
User guide: Update the section on reading math to recommend MathCAT, and also clarify some information about MathPlayer. #16062
Conversation
…and also clarify some information about MathPlayer.
@michaelDCurran , I don't see a mention to Neil to notify about this PR. |
These changes generally look good to me. I have just one point to discuss. It seems that NV Access wants to promote MathCAT and not Access8Math. Could you list in this issue (not in the User Guide) all the reasons for this choice and if it is likely to change in the future or not? This would encourage translators to put effort where it is more useful. Thanks. Even if MathCAT is recommended in the User Guide, I think that mentioning Access8Math too would be useful, indicating that there are different supported languages with respect to MathCAT. Thanks. |
Isn't this Pr also a fix for #15352? |
@lukaszgo1 actually yes it fixes that issue as well. Thank you Mick for taking this up. :-) |
Changes look good so far. One small change: Line 932 reads: "this is only referring to using mathPlayer to display mathematical content visually, and can be ignored by those using it to read or navigate math with NvDA." - there should be a capital at the start of the line and the "v" in NVDA should be capitalised. @CyrilleB79 Not speaking for @michaelDCurran but I suspect the original intention was to encourage use of MathCat over Mathplayer, rather than specifically excluding Access8Math. Would you like to propose an amended wording to include Access8Math for the use cases it is best suited to? |
…nerally recommend any math NvDA add-on, rather than specifically calling out MathCAT. Also move the Installing MathPlayer section to the bottom of the Math section.
@CyrilleB79 To be completely honest, I was completely unaware of Access8math. But having now tested it a bit, it is definitely worthy of being included in the user guide. Thus I have rewritten the section to mention both MathCAT and Access8math. And to Generally recommend one of those newer NVDA add-ons, over the older MathPlayer, rather than the section generally recommending MathCAT specifically. Does this read better now? |
Yes, very nice. Thanks. I have one last question: The URL for Access8Math is https://addons.nvda-project.org/addons/access8math.en.html. @michaelDCurran, any update regarding the community website? Can add-on authors rely on it for NVDA 2024.1? Or do you already recommend to use an alternative URL? @tseng Woody, do you plann to change the documentation URL for the next release of Access8Math? You may wait for Mick's answer to decide. |
I would recommend that add-on authors move their home page URLs away
from the old add-on website at this stage.
Message ID: ***@***.***>
|
Co-authored-by: Sean Budd <sean@nvaccess.org>
…old add-ons website but is kept up to date.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That all reads well, good work everyone.
Sorry, I remember now why I have kept the link on the community website rather than a link to GitHub for my own add-ons. The GitHub's page has the same content as the English readme. However, the community website has translated content for non-English users. For now, we can keep the link on GitHub, given the lack of visibility on the community website.
I have not seen any notification that it has happened since then. When will this happen? Since authors will update their add-ons for NVDA 2024.1 compatibility, I hope this happens before 2024.1rc1, i.e. before add-on authors are able to release stable add-ons compatible with NVDA 2024.1. Thanks. |
I am sorry for my delayed response to this discussion regarding the inclusion of Access8Math in the NVDA User Guide. It was only today, as I began updating Access8Math for compatibility with NVDA 2024.1, that I stumbled upon the updated User Guide and this discussion. |
…and also clarify some information about MathPlayer. (nvaccess#16062) Fixes nvaccess#16036 Fixes nvaccess#15352 Summary of the issue: When reading the section of the User guide about Math support, and then the MathPlayer page from Wiris, it is unclear as to whether MathPlayer is actually needed for NvDA unless you are using an older browser such as Internet Explorer 8. Also, Nthe User Guide does not recommend let alone mention, MathCAT, which is now the replacement for the older and unmaintained MathPlayer. Description of user facing changes Update the math section in the User guide to mention and recommend MathCAT, and clarify that MathPlayer is still relevant on newer browsers.
Link to issue number:
Fixes #16036
Fixes #15352
Summary of the issue:
When reading the section of the User guide about Math support, and then the MathPlayer page from Wiris, it is unclear as to whether MathPlayer is actually needed for NvDA unless you are using an older browser such as Internet Explorer 8.
Also, Nthe User Guide does not recommend let alone mention, MathCAT, which is now the replacement for the older and unmaintained MathPlayer.
Description of user facing changes
Update the math section in the User guide to mention and recommend MathCAT, and clarify that MathPlayer is still relevant on newer browsers.
Description of development approach
n/a
Testing strategy:
Built the user guide as html.
Known issues with pull request:
None known.
Code Review Checklist: