Guard :popover-open selector usage for Firefox compatibility #190

Open
opened 2026-11-02 15:10:28 +01:00 by retoor · 0 comments
Owner

Summary

Clicking the Refactor button on /game does nothing and no confirmation popup appears. Console shows Document.querySelectorAll: ':popover-open' is not a valid selector originating from closeOtherPopovers() in AppDialog.js, called via open(). Same unguarded :popover-open use exists in close() via this.overlay.matches(":popover-open").

Steps to Reproduce

  1. Open /game in Firefox.
  2. Click the Refactor button.
  3. Observe no confirmation popup appears and check console for DOMException.

Expected Behaviour

Refactor confirmation dialog opens.

Actual Behaviour

No confirmation appears. Exception in closeOtherPopovers aborts open() before showing dialog.

Environment

Not provided.


Reported by D-04got10-01 via DevPlace.

## Summary Clicking the Refactor button on /game does nothing and no confirmation popup appears. Console shows `Document.querySelectorAll: ':popover-open' is not a valid selector` originating from `closeOtherPopovers()` in AppDialog.js, called via `open()`. Same unguarded `:popover-open` use exists in `close()` via `this.overlay.matches(":popover-open")`. ## Steps to Reproduce 1. Open /game in Firefox. 2. Click the Refactor button. 3. Observe no confirmation popup appears and check console for DOMException. ## Expected Behaviour Refactor confirmation dialog opens. ## Actual Behaviour No confirmation appears. Exception in `closeOtherPopovers` aborts `open()` before showing dialog. ## Environment Not provided. --- *Reported by **D-04got10-01** via DevPlace.*
Sign in to join this conversation.
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: retoor/devplacepy#190