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.

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

Range Slider V2.0 #1055

Closed
wants to merge 53 commits into from
Closed

Range Slider V2.0 #1055

wants to merge 53 commits into from

Conversation

QuantumDeveloper
Copy link
Contributor

What was fixed:

What was added:

  • Improve RangeSlider events #856 - DragStared/DragDelta/DragCompleted for each of 3 thumbs
  • events for lower/upper value changed
  • OldValues in event args for rangeChanged event
  • Vertical orientation support
  • MoveToPoint feature like in Slider
  • SmallChange/LargeChange - when MoveToPoint = False thumbs will move on the value you set in Small/LargeChange
  • IsSnapToTickEnabled feature with TickFrequency. If set to true, thumbs will snap to ticks.
  • TickBars and tickplacement property for displaying/hiding ticks (Change its ticks width according to minimum and maximum values changed)
  • ExtendedMode property. If it set to false, you cannot do any manipulation inside range except moving thumbs closer/farther to each other, but if it enabled you can use MoveToPoint or Small/Large change inside range by clicking Left mouse button + left or right control button to move left thumb and right mouse button+ left or right control button to move right thumb inside range.
    Using Exteded mode you also can without problems move whole range by clicking leftmouse button
  • MoveWholeRange property will let you move whole range when using MoveToPoint or Small/Large change (working also inside range)
  • MinRangeWidth property sets minimum width of central Thumb. It can be in range from 0 to range slider width/2.
  • AutoToolTipPlacement - will display tooltip, which will move with Thumb and display current value. Implemented for left/central/right thumbs
  • AutotoolTip precision - set the number of digits, which will be shown after . in autotooltip.
  • XAML part of control was rewrited to support this changes.

michaelmairegger and others added 30 commits January 9, 2014 22:09
Add event handler for each of both events (Lower/Upper value changed)
Add event handler for each of both events (Lower/Upper value changed)
…imum update Lower and Upper values now checked for correctness. That leads rangslide width to calculate correctly.

This issue happen if lower and upper valued were not set by user.
Coerce minimum and maximum now return value without chek (temporary solution)
Width calculation fixed.
Coerce Lower/Upper value change check from < to <= and from > to >= to fix issue with incorrect setting Lower or Upper Value if this value = Minimum or Maximum.
…. from 0 to max width.

Deleted overriding of Orientation property because its not needed anymore
Temporary comment all coerce checks because they do`nt work correctly. Needs to be fixed further.
…g left mouse button for lwft thumb and right mouse button for right thumb
Left mouse button behavior: immediate move to muse position on left or right side outside tha range/ also move selected range
Left mouse click + leftControl inside range = move left thumb to cursor position
Right mouse click + leftControl inside range = move right thumb to cursor position
…properties, which can lead to an incorrect behavior of RangeSlider itself
1. OldValue to rangeChanged eventargs
2. TemplateBinding for ticks and Minimum/maximum values for correct ticks caclculation
3. Coerce Upper/Lower/Minimum/Maximum values
4. Interval property
5. Various fixes
…led = true and MinRange>0

2. Locked Recalculationg of values if they riched borders (fixing extra changed event calls)
Start with coerce callbacks
…nding to precision like in standard slider when move with small/large change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants