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

report subscripts and superscripts #10919: exclude reporting on elements in browsers such as buttons etc. #11078

Closed
Adriani90 opened this issue May 1, 2020 · 15 comments · Fixed by #11897
Milestone

Comments

@Adriani90
Copy link
Collaborator

Adriani90 commented May 1, 2020

Steps to reproduce:

  1. Open NVDA and enable reporting of subscripts and superscripts in NVDA's document formating settings
  2. Open a browser i.e. Firefox
  3. While navigating on websites with arrow keys or quick commands in browse mode, notice what NVDA reports (i.e. on some buttons in the comment area of this issue)

Actual behavior:

NVDA reports very often "base line"

Expected behavior:

NVDA should report "base line" only in mathematical content or if this is part of a text block. Not on elements such as buttons etc.

System configuration

NVDA installed/portable/running from source:

Installed

NVDA version:

Version: alpha-20086,f436d4d7

Windows version:

Windows 10 1909 Update

Name and version of other software in use when reproducing the issue:

All commonly used browsers, newest versions.

Other information about your system:

Other questions

Does the issue still occur after restarting your computer?

yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

yes, the version before merging #10919 but this had not have separate reporting for subscripts and superscripts.

If addons are disabled, is your problem still occuring?

yes

Did you try to run the COM registry fixing tool in NVDA menu / tools?

n/a

@feerrenrut
Copy link
Contributor

Other than in MathML or other explicitly defined math, it could be difficult to determine "math content". I think a reasonable path forward is to only announce base line if the last text announced was superscript or subscript. Examples could help to clarify your issue description:

Given:
This is a regular sentence with some superscript note followed by more text

Expected:
Speech: "This is a regular sentence with superscript some superscript note baseline followed by more text"

@feerrenrut
Copy link
Contributor

Interestingly, I can't get NVDA to recognise that "some superscript note" is actually formatted as superscript. It was written like this in the comment, and visually appears as superscript. Pressing NVDA+F on this text announces it as baseline.

This is a regular sentence with <sup>some superscript note</sup> followed by more text

@ultrasound1372
Copy link

Glad I'm not the first to report the issue just referenced, but check there for a good example and breakdown of the behavior. Discussion should continue here though.

@CyrilleB79
Copy link
Collaborator

@Adriani90 and @ultrasound1372, would #11897 allow to close this issue?
The issue with spurious reporting of "baseline" should be fixed but the implemented solution is not to exclude some web elements for reporting.

@CyrilleB79
Copy link
Collaborator

Here is a link to download the test build.

@CyrilleB79
Copy link
Collaborator

@ultrasound1372 and @Adriani90 could you try the test build and indicate if it meets your needs? Would #11897 solve the root cause of this issue?

Note however that it may not solve the issue raised by @feerrenrut regarding <sup> tags in GitHub comments described in #11078 (comment)

@CyrilleB79
Copy link
Collaborator

Ping @Adriani90, @ultrasound1372
Please provide feedback.

@ultrasound1372
Copy link

@CyrilleB79 appears to work for the use case I presented in my issue.

@CyrilleB79
Copy link
Collaborator

@CyrilleB79 appears to work for the use case I presented in my issue.

Yes @ultrasound1372, I have made a PR that may solve this issue.
And that's why I was asking you to test the test build to know if the issue is fixed in this build.
Thanks.

@ultrasound1372
Copy link

Yes, I was saying I tested the test build and it fixes the issue I reported.

@CyrilleB79
Copy link
Collaborator

@Adriani90 it is a last attempt to ask you to test this updated build of PR #11897.
Let me know if you consider that it resolves this issue, even if the fimplemented solution is not what you suggest in this issue.

@Adriani90
Copy link
Collaborator Author

This is still reproducible in NVDA alpha-23852,df92ef66.
Especially in browse mode. I have updated the description accordingly.
Also when you move to the address bar with f6 and press ctrl+left arrow, NVDA reports "base line".

I can reproduce this issue now only in Firefox, version 92 currently.

cc: @jcsteh

@CyrilleB79
Copy link
Collaborator

@Adriani90 my question was not to test last alpha but the build of PR #11897 (cf. my previous message). Could you give a try and see if it resolves this problem for you?

@Adriani90
Copy link
Collaborator Author

Oops I was missing that one.
Testing with Version: pr11897-23840,ffbb45e7, the problem does not occur anymore. Thank you so much for your work!

Sory for the confusion, and that it took so long. I am quite busy these times and don't find much time to contribute to NVDA.

@CyrilleB79
Copy link
Collaborator

No problem, do not worry. The PR has not been delayed because of you but because of me anyway. Thank you for your feedback.

feerrenrut pushed a commit that referenced this issue Feb 15, 2022
Fixes #11815
Fixes #11078.

Summary:
When fetching formatting information with NVDA+F (once or twice) in a Firefox page, 'baseline' was reported.
Only 'subscript' or 'superscript' is expected to be reported, not 'baseline' since 'baseline' is the default text position.
When the 'Report superscript and subscript' option is checked, navigating in a Firefox page cause spurious reporting of 'baseline'. This is due to transitions between elements having no formatting information such as button or graphics and baseline text.

Description of change:
All objects generating the textInfo format field provide only the introduced enum standard values for 'text-position' field:
- superscript
- subscript
- baseline
- unspecified

Unspecified is used in the following cases:
- The API does not provide formatting info (e.g. buttons in Firefox)
- Mixed text-position information, e.g. an Excel cell containing baseline and superscript.
@nvaccessAuto nvaccessAuto added this to the 2022.1 milestone Feb 15, 2022
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 a pull request may close this issue.

5 participants