fix: docs related sliders overflow #3852
Conversation
Stars22
commented
Sep 11, 2023
Stars22
commented
- fixes related slider overflow: when the top slider value(set to min) becomes greater then the max value of bottom sliders close VaSlider goes overflow #3819
| @@ -1,12 +1,12 @@ | |||
| <template> | |||
There was a problem hiding this comment.
yep, my bad. missed the second slider. fixed
There was a problem hiding this comment.
Ok, so we fixed the docs example. We didn't fix the underlying issue.
Range should be operational even if min/max are swapped. It's reasonable to expect user would do something similar in app.
We could either fix it in this PR or move to another issue, but this has to be addressed.
There was a problem hiding this comment.
ok. I think it's right to have fix in this pr. I'll check what we can do.
There was a problem hiding this comment.
I've created a possible solution, I can't say how it's optimal as it was not a trivial task to begin with )
There was a problem hiding this comment.
story is kind of draft not intended for merge, maybe will be helpful to play around with sliders
Stars22
force-pushed
the
fix/docs-slider-overflow
branch
from
238b2d6
to
8c2f3ab
Compare
Stars22
force-pushed
the
fix/docs-slider-overflow
branch
from
8c2f3ab
to
bda138d
Compare
Stars22
force-pushed
the
fix/docs-slider-overflow
branch
from
f4fff5a
to
2359969
Compare
