Repo cleanup - #3899
Conversation
Michael Hoffman (mikehoffms)
added
in-progress
cat: webview2
Michael Hoffman (mikehoffms)
changed the title
Michael Hoffman (mikehoffms)
changed the title
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 83dbaac: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. microsoft-edge/webview2/release-notes/sdk/1-0-nnnn-nn.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
|
Learn Build status updates of commit 6abcd9d: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. microsoft-edge/webview2/release-notes/sdk/1-0-nnnn-nn.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 75e93f9: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
|
||
| * Fixed an issue that could cause WebView2 apps to crash during startup. | ||
|
|
||
| * Fixed a cross-origin redirect failure with wrr<!-- todo: clarify --> attached. |
There was a problem hiding this comment.
Clarify "Fixed a cross-origin redirect failure with wrr attached."
Michael Hoffman (mikehoffms)
changed the title
|
Learn Build status updates of commit 3769ec1: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. |
Michael Hoffman (mikehoffms)
added
cat: general
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
|
||
| | Scenario | By navigating to a file URL | By navigating to an HTML string | By using virtual host name mapping | By using `WebResourceRequested` | | ||
| | --- |:---:|:---:|:---:|:---:| | ||
| |---|:---:|:---:|:---:|:---:| |
There was a problem hiding this comment.
Accessibility review: Change from emojis to one of the following?
- Yes | No
- Yes | --
- Yes | n/a
How does screen reader handle emoji in cell?
The only tables that use center-align instead of the default left-aligned, are dubious re: accessibility.
| @@ -131,52 +151,51 @@ On WinUI 2 (UWP), some of the predefined CSS cursors are not supported. You can | |||
|
|
|||
| | Keyword | Supported? | | |||
| |---|:---:| | |||
There was a problem hiding this comment.
Accessibility review: Change from emojis to one of the following?
- Yes | No
- Yes | --
- Yes | n/a
How does screen reader handle emoji in cell?
The only tables that use center-align instead of the default left-aligned, are dubious re: accessibility.
|
Learn Build status updates of commit 5c8482f: ✅ Validation status: passed
This comment lists only the first 25 files in the pull request. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
1 participant
Rendered pages for review:
Global changes
ms.date:value in all wv2 Runtime relnotes pages & wv2 SDK relnotes pages.description:field.Release notes for the WebView2 Runtime
/webview2/release-notes/runtime/index.mdRelease notes for WebView2 > See also
/webview2/release-notes/index.mdRelease notes for WebView2 SDKs
/webview2/release-notes/sdk/index.mdTOC
toc.yml/scripts/beta-release-notes-template.njkhttps://learn.microsoft.com/microsoft-edge/web-platform/release-notes/152
AB#63791953