diff options
| author | Christophe Besson <cbesson@gmail.com> | 2026-09-21 19:53:33 +0200 |
|---|---|---|
| committer | Christophe Besson <cbesson@gmail.com> | 2026-09-21 19:53:33 +0200 |
| commit | 9f3445d03f106ee3ebd8b4b1bd546a08d9169af7 (patch) | |
| tree | fabae2b4c9e2fee75a73c5ba1479075be908a584 /packages/meshbay-hub/tests/test_playlist_ui.py | |
| parent | 24b563d0073849d0ba74c3fce9822a70c98e9d9f (diff) | |
| download | meshbay-9f3445d03f106ee3ebd8b4b1bd546a08d9169af7.tar.gz | |
fix: ask in the page instead of native confirm/alert
A native confirm() or alert() leaves the desktop client unable to type
until the window is refocused. ask.js draws both in the page; the SPA
test now bans all three browser dialogs.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Diffstat (limited to 'packages/meshbay-hub/tests/test_playlist_ui.py')
| -rw-r--r-- | packages/meshbay-hub/tests/test_playlist_ui.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/packages/meshbay-hub/tests/test_playlist_ui.py b/packages/meshbay-hub/tests/test_playlist_ui.py index c449dc7..8259088 100644 --- a/packages/meshbay-hub/tests/test_playlist_ui.py +++ b/packages/meshbay-hub/tests/test_playlist_ui.py @@ -101,8 +101,7 @@ def test_removing_a_track_removes_that_one(steps): def test_deleting_a_playlist_asks_first(steps): """A deletion is a tombstone: there is nothing in the interface that undoes - it. `confirm` and not a component — Electron implements it and a dozen - places in this SPA already use it.""" + it. Asked in the page, not by `confirm` (`test_no_native_dialogs_in_the_spa.py`).""" s = steps["deleted"] assert s["asked"] is True, "a playlist was deleted without asking" assert s["lists"] == [] |