ethpandaops / dora

Dora the Explorer is a lightweight slot explorer for the ethereum beaconchain
https://beaconlight.ephemery.dev/
GNU General Public License v3.0
74 stars 25 forks source link

client pages: multiple improvements #117

Closed skylenet closed 3 weeks ago

skylenet commented 1 month ago
pk910 commented 3 weeks ago

https://github.com/user-attachments/assets/379359e2-753e-4b68-95ac-2026f435980d

The zoom stepping feels way too high, these are single "scroll steps" on my side :D Also the sizing of some layout feels a bit weird

skylenet commented 3 weeks ago

The zoom stepping feels way too high, these are single "scroll steps" on my side :D Also the sizing of some layout feels a bit weird

Yeah, this is weird indeed. It should have an acceptable scrolling sensitivity by leaving it at the default value (wheelSensitivity=1). I've played around with this in the past and made it work fine for my mouse, but then it wouldn't feel right anymore on a touchpad. But it wasn't as extreme as your example.

It's also not recommended to set it as per CytoScapeJS docs:

wheelSensitivity: Changes the scroll wheel sensitivity when zooming. This is a multiplicative modifier. So, a value between 0 and 1 reduces the sensitivity (zooms slower), and a value greater than 1 increases the sensitivity (zooms faster). This option is set to a sane value that works well for mainstream mice (Apple, Logitech, Microsoft) on Linux, Mac, and Windows. If the default value seems too fast or too slow on your particular system, you may have non-default mouse settings in your OS or a niche mouse. You should not change this value unless your app is meant to work only on specific hardware. Otherwise, you risk making zooming too slow or too fast for most users.

Not sure how to fix this right now, as I'm not able to reproduce it. It works fine on all browsers on MacOS (Firefox, Chrome, Safari) and also on my iPhone (Safari).

This page is a good test: https://cpstest.org/scroll-test.php . In my case, I'm doing 4px scrolling with a single mousewheel step. It then increases gradually as I keep scrolling.

pk910 commented 3 weeks ago

The zoom stepping feels way too high, these are single "scroll steps" on my side :D Also the sizing of some layout feels a bit weird

Yeah, this is weird indeed. It should have an acceptable scrolling sensitivity by leaving it at the default value (wheelSensitivity=1). I've played around with this in the past and made it work fine for my mouse, but then it wouldn't feel right anymore on a touchpad. But it wasn't as extreme as your example.

It's also not recommended to set it as per CytoScapeJS docs:

wheelSensitivity: Changes the scroll wheel sensitivity when zooming. This is a multiplicative modifier. So, a value between 0 and 1 reduces the sensitivity (zooms slower), and a value greater than 1 increases the sensitivity (zooms faster). This option is set to a sane value that works well for mainstream mice (Apple, Logitech, Microsoft) on Linux, Mac, and Windows. If the default value seems too fast or too slow on your particular system, you may have non-default mouse settings in your OS or a niche mouse. You should not change this value unless your app is meant to work only on specific hardware. Otherwise, you risk making zooming too slow or too fast for most users.

Not sure how to fix this right now, as I'm not able to reproduce it. It works fine on all browsers on MacOS (Firefox, Chrome, Safari) and also on my iPhone (Safari).

This page is a good test: https://cpstest.org/scroll-test.php . In my case, I'm doing 4px scrolling with a single mousewheel step. It then increases gradually as I keep scrolling.

yea, let's use it like this for now and revisit later if there are more complains about it :)