docs: Microsoft Edge compatibility notes and smoke checklist - #21
docs: Microsoft Edge compatibility notes and smoke checklist#21YRWoods wants to merge 5 commits into
Conversation
Code-level Edge notes (edge:// restricted, MV3) + manual checklist. Live Edge UI smoke marked untested — no invented pass/fail.
TuYv
left a comment
There was a problem hiding this comment.
Thanks for separating code-level facts from unverified claims. This cannot close #15 yet because the document explicitly leaves the live Edge smoke test untested, while the issue requires exact Edge/OS/Pounce versions and actual results for installation, search, popup actions, options persistence, and restricted pages.
Please run the checklist in current Microsoft Edge and commit the results. If live verification is not available, remove Fixes #15 and keep the issue open.
The restricted-page expectation also needs correction: for protected URLs such as edge://, background.js deliberately opens Pounce's bridge.html overlay; the restricted alert is fallback behavior. Please document and test the actual bridge behavior, expand the popup/options checks to the concrete repository flows, and remove the trailing whitespace reported by git diff --check.
|
Addressed review: bridge.html documented as primary protected-page path; concrete flows; #15 stays open until live Edge table filled (no invented pass/fail). Trailing whitespace cleaned. |
|
Ready for re-review: bridge.html documented as primary protected-page path; concrete smoke flows; #15 not auto-closed until live Edge table is filled. |
|
Re-review ready: live Edge smoke matrix filled (Edge 150.0.4078.65, Win11, Pounce 1.6.1). bridge.html path documented as primary for protected pages. Node tests 128/128. Please re-review for #15 close. |
|
Withdrawing. |
Summary
Microsoft Edge compatibility notes for #15 with filled live Edge smoke table.
Documented
edge://+chrome://restricted schemes, bridge.html primary path for protected pagesFixes
Closes #15 (live matrix filled; maintainer may re-verify Alt+K manually if desired).
Test plan
node --test tests/*.test.js→ 128 pass