On mobile, opening a tall dialog (e.g. the replace/add-vdev one) anchors it at the bottom of the screen. Its action button ends up out of view, so the user has to scroll the dialog body. That gesture bubbles to the global pull-to-refresh handler (touchmove on window with scrollY<=0), which accumulates the pull and calls window.location.reload() once the threshold is crossed. Result: the page reloads and the dialog (and its action button) disappears before the user can tap it.
Steps to reproduce:
- Open the app on a mobile viewport.
- Open a pool action dialog whose content is taller than the viewport (e.g. replace disk).
- Swipe up to scroll the dialog content down to the action button.
- The pull threshold is crossed and the whole page reloads, closing the dialog.
Expected: scrolling inside a modal must not trigger pull-to-refresh; only a pull from the page top with no modal open should reload.
The pull-to-refresh handler attaches to window and never checks whether the touch originated inside the modal overlay.
On mobile, opening a tall dialog (e.g. the replace/add-vdev one) anchors it at the bottom of the screen. Its action button ends up out of view, so the user has to scroll the dialog body. That gesture bubbles to the global pull-to-refresh handler (touchmove on window with scrollY<=0), which accumulates the pull and calls window.location.reload() once the threshold is crossed. Result: the page reloads and the dialog (and its action button) disappears before the user can tap it.
Steps to reproduce:
Expected: scrolling inside a modal must not trigger pull-to-refresh; only a pull from the page top with no modal open should reload.
The pull-to-refresh handler attaches to window and never checks whether the touch originated inside the modal overlay.