Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: Horizontal scrolling #7134

Open
thibaultmol opened this issue Dec 18, 2019 · 4 comments
Open

Feature request: Horizontal scrolling #7134

thibaultmol opened this issue Dec 18, 2019 · 4 comments
Labels
compatibility An issue with hardware or software support considering Not Actionable - still considering if this is something we want enhancement An enhancement to make an existing feature even better usability An issue with ease-of-use or design

Comments

@thibaultmol
Copy link
Contributor

Hi,

I recently realized that being able to move the slippy map using the horizontal scrollwheel that a lot of mice have these days, could be useful.
I imagine using it like this:

Horizontal scrolling: moves the map horizontally.
SHIFT + Horizontal scrolling: moves the map vertically

(right now horizontal scrolling doesn't do anything in the editor it seems)

Cheers,
Thibault

@quincylvania
Copy link
Collaborator

@thibaultmol Hmm scroll-to-pan already works on many devices. Perhaps this is #6598?

@quincylvania quincylvania added the waitfor-info Waiting for more info label Dec 18, 2019
@thibaultmol
Copy link
Contributor Author

thibaultmol commented Dec 18, 2019

Just tried it on my windows pc with chrome stable. Also doesn't work.
I don't think they're related though.
Just so we're on the same page, I'm talking about these horizontal scroll wheels:
image

Example code:
https://jsfiddle.net/alecazam/my6gntrr/
See video of that jsfiddle and me scrolling on the horizontal scrollwheel: https://photos.app.goo.gl/ZXNx18W3pKhagG9dA

It can be implemented using 'wheelDeltaX', see this documentation: https://developer.mozilla.org/en-US/docs/Web/API/Element/mousewheel_event

@quincylvania quincylvania added usability An issue with ease-of-use or design enhancement An enhancement to make an existing feature even better and removed waitfor-info Waiting for more info labels Mar 5, 2020
@quincylvania quincylvania added compatibility An issue with hardware or software support and removed usability An issue with ease-of-use or design labels Sep 14, 2020
@tyrasd tyrasd added the usability An issue with ease-of-use or design label Feb 3, 2022
@tyrasd
Copy link
Member

tyrasd commented Feb 18, 2022

I tried this suggestion out today (in the issue-7134-horizontal-scrollwheel branch), and it works in principle.
But to be honest, while trying it out, I found it not very useful in practical usage. It's pretty rare that one only wants to pan horizontally or vertically and while mapping long linear features (such as a stream for example) click-dragging is more useful since it does not modify the relative position of the mouse pointer to the already-drawn part of the feature.

It also doesn't help that the regular "Y" scroll wheel has a different functionality already which means that in order to navigate the map using the horizontal "X" scroll wheel alone requires the use of modifier keys (which feels also quite clumsy).

@thibaultmol: Did you have a specific use case in mind for panning via the horizontal scroll wheel, which I'm not seeing at the moment?

Otherwise, I think there are better ways to utilize the horizontal scroll wheel in iD, for example to cycle through different background layers, cycle through the nodes of a way and/or to nudge/scale the position of existing features.

@tyrasd tyrasd added the considering Not Actionable - still considering if this is something we want label Feb 18, 2022
@thibaultmol
Copy link
Contributor Author

I had a while where I accidentally moved objects while panning by just dragging.

Problem is, I don't know where my MX Master is, so i can't actually test this branch atm.
What do you mean with Y scrollilng btw?

(If you think it's not a good addition to iD, then I guess you could close the issue for now (I can always reopen it when I get my hands on a mouse again with and extra scroll wheel like that... I'm sure the MX Master is lying around here somehwere... we'll find it.. XD)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compatibility An issue with hardware or software support considering Not Actionable - still considering if this is something we want enhancement An enhancement to make an existing feature even better usability An issue with ease-of-use or design
Projects
None yet
Development

No branches or pull requests

3 participants