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

[General Question] Accessibility Insights + WebView2 app #1623

Closed
kiddliu opened this issue May 29, 2023 · 6 comments
Closed

[General Question] Accessibility Insights + WebView2 app #1623

kiddliu opened this issue May 29, 2023 · 6 comments
Assignees

Comments

@kiddliu
Copy link

kiddliu commented May 29, 2023

With the latest production update, the tool reports lots of 'Chromium components should be scanned with a web-based scanner' after scanned a WebView2-based application, and in the HOW TO FIX section it recommendeds Accessibility Insights for Web, which is a extension to Chrome/Edge.

Questions:

  1. Should I continue to run tests against WebView2 applications with the Windows tool? Should I stay with the last good version?
  2. Despite the tons of warnings, are other reported issues still valid ones?
  3. Is there any workaround for the warning?
@microsoft-github-policy-service microsoft-github-policy-service bot added the status: new This issue is new and requires triage by DRI. label May 29, 2023
@DaveTryon
Copy link
Contributor

@kiddliu, we made this change because Accessibility Insights for Windows can basically say "Something is wrong with this HTML app", but it can't tell you what to do about it. Accessibility Insights for Web understands HTML and can point out the specific problem. If you can render the HTML in a standalone browser where Accessibility Insights for Web is installed, you'll end up with far more actionable results.

Errors outside of a Chromium control are still valid and should be fixed. Running an old version will work for a short time, but we only support N-1 version, that option will go away when we ship the next release.

@DaveTryon DaveTryon added status: needs author feedback This issue requires additional information from the issue author. and removed status: new This issue is new and requires triage by DRI. labels May 30, 2023
@microsoft-github-policy-service
Copy link
Contributor

The team requires additional author feedback; please review their replies and update this issue accordingly. Thank you for contributing to Accessibility Insights!

@DaveTryon DaveTryon assigned DaveTryon and kiddliu and unassigned JGibson2019 May 30, 2023
@kiddliu
Copy link
Author

kiddliu commented May 31, 2023

Hey @DaveTryon, WebView-based applications usually have critical dependencies on the injected native objects and it may require a huge effort to make it renderable in the browser. Is it possible for the Windows version to support WebView2 apps natively in the future?

@microsoft-github-policy-service microsoft-github-policy-service bot added status: needs attention This issue requires DRI's attention. and removed status: needs author feedback This issue requires additional information from the issue author. labels May 31, 2023
@DaveTryon
Copy link
Contributor

@kiddliu, that's an intriguing idea! Realistically, though, it's probably way beyond the scope of what our team is able to take on. I could envision a scenario where an app hooks the loader, then runs an HTML scanner (either axe-core or https://www.npmjs.com/package/accessibility-insights-scan) to capture the results. It probably would require an instrumented build to connect the scanner to the load hook. You're free to file a feature request at https://github.com/microsoft/accessibility-insights-web/issues/new/choose, but our team's backlog is already stretched pretty thin, so there are no guarantees that we'd be able to pick it up.

@DaveTryon DaveTryon added status: needs author feedback This issue requires additional information from the issue author. and removed status: needs attention This issue requires DRI's attention. labels May 31, 2023
@microsoft-github-policy-service
Copy link
Contributor

The team requires additional author feedback; please review their replies and update this issue accordingly. Thank you for contributing to Accessibility Insights!

@kiddliu
Copy link
Author

kiddliu commented Jun 1, 2023

Will give the scan CLI a try, it looks quite promising. Thank you, @DaveTryon

@kiddliu kiddliu closed this as completed Jun 1, 2023
@microsoft-github-policy-service microsoft-github-policy-service bot added status: needs attention This issue requires DRI's attention. and removed status: needs author feedback This issue requires additional information from the issue author. labels Jun 1, 2023
@DaveTryon DaveTryon removed the status: needs attention This issue requires DRI's attention. label Jun 17, 2023
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

No branches or pull requests

3 participants