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

Slider (noUiSlider) feature "Merging overlapping tooltips" not working #12558

Closed
ghost opened this issue Jun 9, 2022 · 1 comment · Fixed by #12777
Closed

Slider (noUiSlider) feature "Merging overlapping tooltips" not working #12558

ghost opened this issue Jun 9, 2022 · 1 comment · Fixed by #12777
Labels

Comments

@ghost
Copy link

ghost commented Jun 9, 2022

Which exact Umbraco version are you using? For example: 9.0.1 - don't just write v9

8.18.4

Bug summary

When using the Slider datatype - and having Range turned on - the box showing the range is misplaced CSS-wise.
An example looks like this in Chrome:
image

Another example in Firefox:
image

The box should be in the middle above the ranges selected - but instead the more you move it, the further away it goes instead ending up overflowed out of the containing div element.

This has been re-tested on a clean 8.18.4 installation.

Specifics

No response

Steps to reproduce

  • Make a data type that uses the "Slider" property editor
    • Example:
    • Enable Range: true
    • Minimum value: -50
    • Maximum value: 50
  • Put the data type on a document type
  • View the document type property in the Content section on the document type you put it on
  • Try to move the two ranges around

Expected result / actual result

When moving the ranges around, you'll see that when it should be merged it will not stay in the middle, but instead move further and further from the middle until at some point it snaps back to being in the middle.

@ghost ghost added the type/bug label Jun 9, 2022
@bjarnef
Copy link
Contributor

bjarnef commented Aug 2, 2022

I can confirm this issue.
It is working in noUiSlider v15.4.0 but after this working the merged tooltip is off: leongersen/noUiSlider#1217

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant