all pops should have scroll disabled for the site #61

New Issue

freakoverse · 2024-10-01T06:45:40Z

freakoverse commented

2024-10-01 06:45:40 +00:00

Since the popup themselves already have scroll, and the site/ has scroll enabled as well, it's resulting in a double-scroll.

When a popup appears, the scroll of should be disabled, and reenabled after the popup disappears/closes.

Popup examples: zap, zap split, login, register, (the eventual reply system popup), etc.

Since the popup themselves already have scroll, and the site/<body> has scroll enabled as well, it's resulting in a double-scroll. When a popup appears, the scroll of <body> should be disabled, and reenabled after the popup disappears/closes. Popup examples: zap, zap split, login, register, (the eventual reply system popup), etc.

enes added the

Priority

Low

label 2024-10-17 08:54:43 +00:00

freakoverse changed title from ~~(low) all pops should have scroll disabled for the site~~ to all pops should have scroll disabled for the site

2024-10-17 11:00:44 +00:00

enes referenced this issue from a commit

2024-10-21 09:45:10 +00:00

fix: disable scroll on popup open

enes referenced a pull request that will close this issue

2024-10-21 09:47:14 +00:00

fix: disable body scroll on popup open #88

enes closed this issue

2024-10-21 09:49:13 +00:00

Sign in to join this conversation.