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.

Sign up for GitHub

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

Jump to bottom

[Run] Touchpad scrolling is too sensitive #5383

Open
LuanVSO opened this issue Jul 31, 2020 · 13 comments
Open

[Run] Touchpad scrolling is too sensitive #5383

LuanVSO opened this issue Jul 31, 2020 · 13 comments
Labels
Area-User Interface things that regard UX for PowerToys External Dependency This bug or feature isn't resolved, but it's following an external work item. Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Status-Blocked We can't make progress due to a dependency or issue

Comments

@LuanVSO
Copy link

LuanVSO commented Jul 31, 2020

ℹ Computer information

  • Windows build number: 19042.421
  • PowerToys version: 0.20.0
  • PowerToy module: Run

📝 Provide detailed reproduction steps (if any)

  1. open pt run on a laptop
  2. input a character to show a big list
  3. scroll with the touchpad

✔️ Expected result

smooth scrolling that obeys windows scrolling speed settings

❌ Actual result

way too sensitive scrolling (can scroll to the bottom of the list with just a few millimeters of movement on the touch pad)

📷 Screenshots
@LuanVSO LuanVSO added the Issue-Bug Something isn't working label Jul 31, 2020
@ghost ghost added the Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams label Jul 31, 2020
@crutkas
Copy link
Member

crutkas commented Jul 31, 2020

did you find it this way in .19?

@crutkas crutkas added Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something Product-PowerToys Run Improved app launch PT Run (Win+R) Window and removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Jul 31, 2020
@LuanVSO
Copy link
Author

LuanVSO commented Jul 31, 2020 •
edited
Loading

no, i don't think so🤔

@ghost ghost added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jul 31, 2020
@saahmedm saahmedm added this to the v1.0 Release milestone Aug 4, 2020
@saahmedm saahmedm removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up labels Aug 4, 2020
@crutkas crutkas mentioned this issue Dec 11, 2020
Closed
@enricogior enricogior changed the title touchpad scrolling is too sensitive on PT run [Run] Touchpad scrolling is too sensitive Jan 9, 2021
@niels9001 niels9001 added the Area-User Interface things that regard UX for PowerToys label Jun 14, 2021
@crutkas
Copy link
Member

crutkas commented Dec 7, 2021

Is this still a thing? I think .20 was still under XAML Islands so this should be smoother.

@LuanVSO thoughts?

@crutkas crutkas added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Dec 7, 2021
@LuanVSO
Copy link
Author

LuanVSO commented Dec 7, 2021

it's still kinda fast for me, it also lacks smooth scrolling

PowerLauncher.2021-12-07.18-44-24.mp4

@ghost ghost added Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams Needs-Team-Response An issue author responded so the team needs to follow up and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Dec 7, 2021
@stefansjfw stefansjfw mentioned this issue Dec 16, 2021
Merged
10 tasks
@stefansjfw stefansjfw added Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. and removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Dec 17, 2021
@stefansjfw
Copy link
Collaborator

Fix is merged. Will be included in next release

@crutkas
Copy link
Member

crutkas commented Jan 6, 2022

The was fixed in in 0.53. Head over to https://aka.ms/installpowertoys to try it out

@crutkas crutkas closed this as completed Jan 6, 2022
@aquinn39
Copy link

0.53 does not fix this for me. I believe the problem is that WPF's ScrollViewer ignores the mouse wheel delta value (see here for more info: dotnet/wpf#5936). Possible solutions might be to use XAMLIslands again like PowerToys run used to or to handle scrolling manually via PreviewMouseWheel event of the WPF ScrollViewer, taking into account MouseWheelArgs.Delta value.

@franky920920
Copy link
Contributor

I think we do revert this in #15420 since the fix for this is causing some crashes.
I'm reopening this.
@aquinn39 Thanks for reporting this!

@franky920920 franky920920 reopened this Jan 17, 2022
@franky920920 franky920920 removed the Resolution-Fix Committed Fix is checked in, but it might be 3-4 weeks until a release. label Jan 17, 2022
@franky920920
Copy link
Contributor

@crutkas Please update the project.

@crutkas
Copy link
Member

crutkas commented Jan 19, 2022

Islands have quirks that makes this style app hard with them (PT Run was initially done with islands believe it or not)

As of now, this is blocked as this is how WPF just works sadly.

@crutkas crutkas added Status-Blocked We can't make progress due to a dependency or issue External Dependency This bug or feature isn't resolved, but it's following an external work item. and removed Needs-Team-Response An issue author responded so the team needs to follow up labels Jan 19, 2022
@crutkas crutkas removed this from the Stability Release milestone Jan 19, 2022
@embender embender mentioned this issue Aug 11, 2023
Closed
@github-actions github-actions bot mentioned this issue Jul 12, 2024
Closed
@hieu-delta
Copy link

v0.84.1
Stills unfixed

@IamSmeagol
Copy link

v0.86.0 still CR

@aquinn39
Copy link

Maybe we can do something like this? dotnet/wpf#5936 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area-User Interface things that regard UX for PowerToys External Dependency This bug or feature isn't resolved, but it's following an external work item. Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Status-Blocked We can't make progress due to a dependency or issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
9 participants
@crutkas @niels9001 @franky920920 @LuanVSO @aquinn39 @stefansjfw @saahmedm @IamSmeagol @hieu-delta