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

MSHTML: Support for CSS content property #4150

Closed
nvaccessAuto opened this issue May 22, 2014 · 9 comments
Closed

MSHTML: Support for CSS content property #4150

nvaccessAuto opened this issue May 22, 2014 · 9 comments
Labels
Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. app/internet-explorer bug feature/browse-mode p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority

Comments

@nvaccessAuto
Copy link

Reported by khsruru on 2014-05-22 12:29
Hello, I’m Hyongsop Kim in Korea.
I have a question about reading css content properity.
By using content properity in css, developer can add a text in css.
But Nvda doesn’t read it in internet explorer.
Because content is a text, I think Nvda should read this text in internet explorer, too.
Please check this issue.
Thank you.

@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2014-05-23 00:02
Confirmed. Test case:

<style>
div:before {
 content: "before ";
}
</style>
<div>div</div>

As usual in IE, we'll have to process the CSS ourselves, since IE doesn't bother to expose this via accessibility APIs. We can get this using window.getComputedStyle(element, ":pseudo"), where pseudo is "before" or "after". It'll be pretty difficult to support values other than a string. You can also have counters, attribute values, etc. and I'm not sure it's worth the effort to support these, at least initially.

More info about the content property

@nvaccessAuto
Copy link
Author

Comment 2 by jteh on 2014-05-23 00:02
Changes:
Changed title from "Nvda and css content properity." to "MSHTML: Support for CSS content property"

@bhavyashah
Copy link

@jcsteh @khsbory Could you please clarify the versions of Internet Explorer the reported issues applies to? In case this has been fixed by any chance in Internet Explorer 10/11, then we may safely close this ticket. Otherwise, we may need to mark this as P4.

@jcsteh jcsteh added p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority app/internet-explorer labels Aug 27, 2017
@jcsteh
Copy link
Contributor

jcsteh commented Aug 27, 2017

STR:

  1. Paste the test case above into an empty HTML file.
  2. Open it in Internet Explorer.
  3. Read the document in browse mode.
    • Expected: "before div"
    • Actual: "div"

Still persists in Internet Explorer Version: 11.0.16251.0, Update Versions: 11.0.41(KB4014661)

@Adriani90
Copy link
Collaborator

Still reproducible in NVDA 2019.1.1. I wonder if someone can test in the IE mode of Edge chromium if this issue occurs there as well?

@LeonarddeR
Copy link
Collaborator

If I"m correct, this requires a change both in the MSHTML virtual buffer and on the MSHTML object level.

While I know that IE is still used in some corporate environments, IE is considered legacy by Microsoft. Given this is an issue that requires a lot of effort to implement, I'm tempted to close this as a wont fix. We can always reopen and remove the label if someone provides a pr.

@lukaszgo1
Copy link
Contributor

@LeonarddeR I have no interest in this particular issue, however my impression is that when issue is closed the PR for it wouldn't be accepted. Leaving it open as a P4 seems more reasonable.

@LeonarddeR
Copy link
Collaborator

LeonarddeR commented Oct 3, 2019 via email

@Adriani90 Adriani90 added the Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. label Apr 14, 2023
@Adriani90
Copy link
Collaborator

Support for IE ended completely in 2022 and is being automatically removed from machines that still have it after an Edge update in February 2023. Even though MSHTML will still be active until 2029 especially for backwards compatibility in IE mode in Edge, I don't expect any accessibility updates to IE or to IE mode in Edge. Closing as abandoned.

@Adriani90 Adriani90 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. app/internet-explorer bug feature/browse-mode p5 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority
Projects
None yet
Development

No branches or pull requests

6 participants