5

mouse: Add mouse "acceleration profile" row

 1 year ago
source link: https://gitlab.gnome.org/GNOME/gnome-control-center/-/merge_requests/1548
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

mouse: Add mouse "acceleration profile" row

Fixes #94 (closed)


It is ultimately a design decision on whether this should be exposed in the UI or not.

image

Also, we can add a description to the row (the screenshot above is merely illustrative, and the text needs work).

image
Pipeline #492094 passed for 7ba1afc6 on mouse-add-accel-profile-row 5 days ago
Approved by

Merged by 5 days ago (Feb 14, 2023 11:17am UTC) 5 days ago

Merge details

Pipeline #492096 passed for 7ba1afc6 on main 5 days ago

Activity

    • Resolved by Michael Catanzaro 2 weeks ago

      I think the description for this could be discussed a bit, I have the following in mind:

      "The "Default" profile lets the mouse select its own acceleration profile, "Flat" is a constant factor, and "Adaptive" changes the acceleration depending on the current mouse movement."

    • Last reply by Michael Catanzaro 2 weeks ago
  • ftr, we're currently working on a custom acceleration profile in libinput, likely to be merged in the next libinput release. That profile however is not useful without significant tuning of multiple parameters. For that profile, a drop-down list like suggested above is not useful. But we'll likely have to ignore that profile here anyway (too niche and too complex to configure).

    The two that are suitable for the control-center are "adaptive" which is effectively "some pointer acceleration" and "flat" which is effectively "no pointer acceleration" (or rather just a constant one). Maybe it's better to have this as a toggle instead. So, as already mentioned something like "Pointer acceleration: pointer speed adapts to mouse movement speed".

    • Resolved by Felipe Borges 1 week ago

      I'm still fairly unconvinced this is Settings material. I don't think this is relevant for most people. Acceleration profiles are hard to explain, and we're probably just going to make people confused. Some more adventurous might try these options to see the results. I feel like this is Tweaks material, not Settings.

    • Last reply by Leaflet 2 weeks ago
    • Resolved by Andre Klapper 2 weeks ago

      Actually, it has come to my attention that this is already available in Tweaks:

      image

      Given that myself and members of the design team share the opinion that this is not a good fit for Settings, I think this is 3. Out of Scope for now. @felipeborges do you have objections to closing this MR?

    • Last reply by Andre Klapper 2 weeks ago
    • Resolved by Michael Catanzaro 2 weeks ago

      A use case to consider: Family member with Parkinson's occasionally use my computer. I prefer the Default Acceleration but this is not desirable if you have hand tremors(1). I end up changing this setting in the Tweak tool more often than I change anything in the Settings tool.

      (1) Ideally a custom acceleration profile to attenuate at tremor frequency would be the best but a flat profile also helps a lot.

    • Last reply by Peter Hutterer 3 weeks ago
  • @aday did I understand correctly that you're open to adding this with the improvements to the name and description?

    @feaneron no objections if Allan is fine with it?

    • Resolved by Felipe Borges 6 days ago

      A use case to consider: Family member with Parkinson's occasionally use my computer. I prefer the Default Acceleration but this is not desirable if you have hand tremors(1). I end up changing this setting in the Tweak tool more often than I change anything in the Settings tool.

      If it's advertised as a accessibility setting, it should probably go in the accessibility panel, along with other "pointing & clicking" settings.

      If the goal is to have it as a general purpose setting, it would be useful to answer Allan's question about why someone might use one profile over the other, especially when apps can already request raw movement information in situations where they would want to control the pointer.

    • Last reply by John Smith 6 days ago
    • Resolved by GitLab Admin - Carlos Soriano 3 weeks ago

      While I would absolutely love to have mouse acceleration options added into settings, this isn't the way of getting that done.

      Please be respectful. While you may not agree with him in this particular topic, he has done a ton of work for the betterment of the Gnome project.

      Edited by Leaflet 1 month ago
    • Last reply by GitLab Admin - Carlos Soriano 3 weeks ago
  • The design team would have preferred not to have included the mouse acceleration setting in with the general mouse & touchpad settings. However, we're yet to identify a viable alternative, and so are willing to include it in the settings panel, on the condition that the description of the setting is intelligible to a general audience.

    On that basis, I've added an initial design for mouse acceleration to the mouse & touchpad mockups.

    Given the contentious nature of this change, I think it's best for any further discussion of the design to happen in a venue that allows a fuller conversation - either on chat in #gnome-design, or in a call, which I'd be happy to set up.

    Please be mindful of the different points of view on this topic, thanks!

    Edited by Allan Day 2 weeks ago
  • Compare with previous version

  • Felipe Borges marked this merge request as ready 1 week ago
  • Compare with previous version

  • Felipe Borges marked this merge request as ready 6 days ago
  • Automatically resolved by Felipe Borges 6 days ago
  • Compare with previous version

  • Felipe Borges enabled an automatic merge when the pipeline for 332055f2 succeeds 5 days ago
  • Felipe Borges aborted the automatic merge because source branch was updated 5 days ago
  • Felipe Borges enabled an automatic merge when the pipeline for 7ba1afc6 succeeds 5 days ago
Please register or sign in to reply
0 Assignees
Reviewer
Milestone
Time tracking
No estimate or time spent
Lock merge request
Locked
23 participants
Reference: GNOME/gnome-control-center!1548

Source branch: mouse-add-accel-profile-row


About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK