Skip to content

MouseWheel properties are equivalent #2791

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

Merged
merged 2 commits into from
Jul 23, 2019
Merged

Conversation

JeremyKuhne
Copy link
Member

Summary

This was never clearly documented. We supported IntelliMouse drivers on Windows 9x systems. NativeMouseWheelSupport told you if the OS natively had a mouse wheel installed. MouseWheelSupport told you if the OS had a mouse wheel installed OR an IntelliMouse driver was installed to give said support. The driver sent mouse wheel messages on 9x systems, but the OS didn't natively understand what that was.

Also removing the OS comments as they can be confusing and we're long long past those days. :)

This was never clearly documented. We supported IntelliMouse drivers on Windows 9x systems. `NativeMouseWheelSupport` told you if the OS natively had a mouse wheel installed. `MouseWheelSupport` told you if the OS had a mouse wheel installed OR an IntelliMouse driver was installed to give said support. The driver sent mouse wheel messages on 9x systems, but the OS didn't natively understand what that was.

Also removing the OS comments as they can be confusing and we're long long past those days. :)
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

Thanks for doing this update, @JeremyKuhne. Two of the xrefs don't resolve -- are they intended to be NativeMouseWheelSupport?

@rpetrusha
Copy link

Thanks for correcting the API references, @JeremyKuhne. I'll merge your PR now.

@rpetrusha rpetrusha merged commit 83de65a into dotnet:master Jul 23, 2019
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.

3 participants