Skip to content

Conversation

@teamcons
Copy link
Contributor

@teamcons teamcons commented Dec 4, 2024

elementary's default implementation felt very heavy - you have to scroll a lot before it picks up anything, and the experience of going immediately a few workspaces left or right feels frustrating
The icon is too slow and keyboard not always in reach, so i use that feature daily.

"1" and "2" allows for a few misinputs (such as when mousewheel is between two notches somehow), but also picks up quickly enough user intent to react and switch
From using it a few days, ive set on "1" as "2" sometimes ignored valid inputs still. (roughly on my mouse thats the amount of notches that can be scrolled without causing a switch- I am not sure if other mouses would be different)

you cannot "scroll" too hard and go more than one workspace because the panel ignores further scrolling until the new workspace is fully shown. So low value is not an issue like it can be in another interpretations ive used, where you scroll a little too hard and end up too far in the workspace "stream" - Here you still end up scrolling one after the other

EDIT: Eventual refinements:
if the workspace switch isnt triggered by a scroll, the panel seems to "remember" the misinput and not accept any future one until it switches again

#588

@teamcons
Copy link
Contributor Author

summoning @elementary/desktop-developers

@alainm23
Copy link
Member

alainm23 commented Feb 1, 2025

I like it, it's faster.

@teamcons
Copy link
Contributor Author

teamcons commented Feb 1, 2025

I like it, it's faster.

pondering whether it wouldnt even be better to remove that limitation, and scroll anytime a scroll is detected at all

@stsdc stsdc requested a review from a team February 2, 2025 18:41
Copy link
Member

@danirabbit danirabbit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested this with both a mouse and a touchpad and this works for me. Sorry for taking so long to merge!

@danirabbit danirabbit merged commit 51b38f8 into elementary:master Jun 3, 2025
4 of 5 checks passed
@teamcons
Copy link
Contributor Author

teamcons commented Jun 3, 2025

tested this with both a mouse and a touchpad and this works for me. Sorry for taking so long to merge!

I did not really expect it to be merged tbh since it could be more of a matter of taste - I felt this was kind of a "lets wait and see if someone else complains about the default"

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.

4 participants