Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Feature optionally disable mouse wheel #56

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

Conversation

jholzer
Copy link
Contributor

@jholzer jholzer commented Jan 10, 2023

Disable change of value using mouse wheel scrolling while hovering over control by bypassing mouse scroll event handler by settings "MouseWheelEnabled" to false.
Default for MouseWheelEnabled: true

Jürgen Holzer added 2 commits January 10, 2023 10:07
…ill be overwritten with "LastEditingNumericValue" if "IsUpdateValueWhenLostFocus" is enabled on lost focus

- Consolidate "FormatText" in base class (it´s nearly the same code in all derived classes)
- introduce abstract "ParseText" and type specific implementations for usage in in "FormatText", which was the only difference in the several implementations#
- return parsed value in "FormatText" to be used in e.g. _PART_TextBox_LostFocus, where it is assgined to "Value" (here´s were "Value" got lost)
- activate "IsUpdateValueWhenLostFocus" in demo application
@Dirkster99
Copy link
Owner

Please add a description to this PR:
1 what bug are fixing? Under what condition does the bug show in the previous version but is fixed in your PR?
2 What mouse wheel feature is being disabled by which new option?

Thanx Dirk

@jholzer
Copy link
Contributor Author

jholzer commented Jan 10, 2023

Please add a description to this PR: 1 what bug are fixing? Under what condition does the bug show in the previous version but is fixed in your PR? 2 What mouse wheel feature is being disabled by which new option?

Thanx Dirk

I´ve updated both PRs with corresponsding description

@Dirkster99 Dirkster99 merged commit 5a7ca5a into Dirkster99:master Mar 24, 2023
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.

2 participants