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

make mouse interaction wrap back #938

Closed

Conversation

nnyyxxxx
Copy link
Contributor

@nnyyxxxx nnyyxxxx commented Nov 12, 2024

Type of Change

  • New feature

Description

makes mouse interaction wrap back on last / first entry when scroll is attempted again in that direction, see video for a better explanation

we have this same feature for regular keyboard movement, btw

resolves #939

Testing

video is pretty laggy, you are warned !!

2024-11-12.05-56-35.mp4

@jeevithakannan2
Copy link
Contributor

jeevithakannan2 commented Nov 12, 2024

Scrolling with mouse is sensitive that's why it was not implemented ( Also why are you adding unnecessary code if you can just call the scroll_up and scroll_down functions ) . This makes the UX experience worse.

Don't want the user to go bazooka if he just wants to select the last command.

2024-11-12.18-32-20.mp4

@nnyyxxxx
Copy link
Contributor Author

it's not sensitive for me

@jeevithakannan2
Copy link
Contributor

it's not sensitive for me

It's generally not implemented anywhere you can take your browser or htop or btop for example

@nnyyxxxx
Copy link
Contributor Author

it's a good QOL feature, i think everyone can agree with me on that + we should keep the synergy with keyboard movement, the user should expect a seamless transition between the two.

@nnyyxxxx
Copy link
Contributor Author

nnyyxxxx commented Nov 12, 2024

anyways in the video your scroll wheel is clearly unlocked.. in the video i provided this is clearly not the case for me as i have a regular scroll wheel and i clearly didn't over exaggerate the movement in the video

@jeevithakannan2
Copy link
Contributor

@ChrisTitusTech I am against this PR read through the comments before merging.

@lj3954
Copy link
Contributor

lj3954 commented Nov 13, 2024

Most people don't expect mouse scrolling to wrap around. I oppose this change. Mouse and keyboard input aren't supposed to function identically, they're very different tools.

@adamperkowski
Copy link
Collaborator

i agree with Liam on this one. to be honest, i'm against mouse functionality in linutil at all. sorry i'm not really active recently, will be back probably around next week.

@nnyyxxxx
Copy link
Contributor Author

to be honest, i'm against mouse functionality in linutil at all.

i agree, lets hope #942 gets merged since Chris also agree'd on discord as well

@nnyyxxxx
Copy link
Contributor Author

or atleast these 2 PRs if not, #938 #937

@nnyyxxxx
Copy link
Contributor Author

i agree with the rest here, i'm going to close this as it will not be compatible with #946

@nnyyxxxx nnyyxxxx closed this Nov 13, 2024
@nnyyxxxx nnyyxxxx deleted the make_mouse_interaction_wrap_back branch November 13, 2024 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mouse interaction should wrap back on last / first entry just like keyboard interaction
4 participants