appbaseio / reactivesearch

Search UI components for React and Vue
https://opensource.appbase.io/reactivesearch
Apache License 2.0
4.89k stars 466 forks source link

Hoping improvements of RangeSlider #837

Closed AlloVince closed 2 years ago

AlloVince commented 5 years ago

Issue Type: enhancement

Platform: Web

Description: As I mentioned in #836, I'm tring to migrate from searchkit to reactivesearch, the searchkit has some good features I think RangeSlider is really needed

First I think when sliding, the range of start, and the end, should allow same value, it is a useful search, it not allow same value, I have to add a new SingleRange component to do this kind of search

Then when dragging the slider handler, I think its really necessary to show a tooltip of current value, as same as searchkit does. If the range of RangeSlider have many values, I've no idea which value it is now, it make this component really hard to use.

2019-02-18 5 34 51
metagrover commented 5 years ago

@jyash97 didn't we add this?

jyash97 commented 5 years ago

Hey @AlloVince You can show tooltip by using prop tooltipTrigger="always". Check this example here.

AlloVince commented 5 years ago

Hey @AlloVince You can show tooltip by using prop tooltipTrigger="always". Check this example here.

Great! it works. it is better to add this param to docs and typescript definition, I didn't find it in both.

Also please consider my start/end suggestion, it is also an important feature.

metagrover commented 5 years ago

hey @AlloVince, we support rendering start/end values on the range slider.

Can you expand on the issue you're seeing with start/end?

AlloVince commented 5 years ago

hey @AlloVince, we support rendering start/end values on the range slider.

Can you expand on the issue you're seeing with start/end?

What I want is allow start and end with same value, for example, here is a Year slider, range from [2002, 2019]

In searchkit component, I could slide to Year range [2019, 2019]

2019-02-19 9 30 06

But in reactivesearch, the Year range could only be [2018, 2019], [2019, 2019] is not allowed

2019-02-19 9 30 19
metagrover commented 5 years ago

Got it! @jyash97 can you take a look at this?

rgb-panda commented 3 years ago

Hi @metagrover, was there any solution to have the same start-end for RangeSlider? At the moment I can only do it with RangeInput, and doing it manually inside start-end input fields but not on the slider

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.