-
Notifications
You must be signed in to change notification settings - Fork 81
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
Porting RadialGauge #45
Conversation
Found a bug - setting the ScalePadding too high causes a crash. Recording.2023-05-09.134102.mp4 |
Definitely love the new position of the ticks, on the inside of the scale. |
The default value for IsInteractive (= reacting to touch and mouse events) was changed from false to true. That's a good idea, but then you also need to register the Tapped and Manipulation* events in the OnApplyTemplate. e.g. (not sure this way is the best practice though, but it works)
Otherwise the gauge will not be interactive, even when set specifically: |
Thanks! Addressed the feedback, is now works as intended :)! |
Before we merge this and publish to NuGet, I need to check that #43 won't be an issue here. |
This PR ports over the
RadialGauge
:RadialGauge
from the existing toolkitScaleLength
,ScaleTickCornerRadius
,NeedleBorderBrush
,NeedleBorderThickness
,TickCornerRadius
,TickPadding
Before:
After: