Skip to content

Conversation

@MathieuSollier
Copy link

I needed to have a vertical slider with increasing values, ie the minimum is at the bottom and the maximum is at the top.

@simeydotme
Copy link
Owner

Hi @MathieuSollier ,
I totally appreciate the sentiment and reason behind this... and after seeing it, I think I might have made a mistake to put the larger numbers at the bottom in Vertical mode. As you can see; https://simeydotme.github.io/jQuery-ui-Slider-Pips/#example-vertical it probably is incorrect. 😬

I then realised that you can achieve the outcome with a simple formatter();
https://svelte.dev/repl/c77ce2e5938f455ca1c20115d91b8d2d?version=3.41.0

Please feel free to borrow that reversing formatter code, for now, and let me consider reversing the orientation for vertical so that the max is at the top. This will probably require a v2.0 release though, as I have no idea how many people may be relying on the current setup.

@simeydotme simeydotme self-assigned this Jul 27, 2021
@simeydotme simeydotme added bug Something isn't working compatibility enhancement New feature or request labels Jul 27, 2021
@simeydotme simeydotme added this to the 2.0.0 milestone Jul 27, 2021
@MathieuSollier
Copy link
Author

MathieuSollier commented Jul 27, 2021 via email

@simeydotme simeydotme closed this in a255891 Oct 2, 2021
@simeydotme
Copy link
Owner

Hi @MathieuSollier ... Please check out https://svelte.dev/repl/6bf5d88d6ece4ba9a26443e33c79f1c0?version=3.43.1 for an example of the changes. I haven't published them on NPM yet as I'm still debating the other change in the 2.0.0 milestone.

If you want to try out these changes, u may directly call 2.0.0 tag from github in your package.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working compatibility enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants