Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

md-slider does not update correctly when change min value #7474

Closed
johnrivelt opened this issue Mar 8, 2016 · 4 comments
Closed

md-slider does not update correctly when change min value #7474

johnrivelt opened this issue Mar 8, 2016 · 4 comments
Assignees
Labels
P4: minor Minor issues. May not be fixed without community contributions.
Milestone

Comments

@johnrivelt
Copy link

#479

It still seems not working correctly in my case.
Here is my case from plunker.

https://embed.plnkr.co/FCizUTQnOTzsklhxSIry/

click SSD 1 -> SSD 10 direction set min and value correctly.
But SSD 10 -> SSD 1 backward case, md-slider does not update correctly.
Even min-value is same with valuenow but slider cursor indicate middle of somewhere.

@topherfangio topherfangio added the needs: demo A CodePen demo or GitHub repository is needed to demonstrate the reproduction of the issue label Mar 17, 2016
@topherfangio
Copy link
Contributor

Can you please create a more simple example using Codepen? There is a lot of custom code in the existing Plunkr that makes it difficult to understand.

@johnrivelt
Copy link
Author

Here is simpler one.

http://codepen.io/johncho/pen/rejKJj

On Fri, Mar 18, 2016 at 8:22 AM, Topher Fangio notifications@github.com
wrote:

Can you please create a more simple example using Codepen? There is a lot
of custom code in the existing Plunkr that makes it difficult to understand.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#7474 (comment)

Regards
John, Cho
王言, 趙
Wang Eon, Cho

@topherfangio topherfangio added P4: minor Minor issues. May not be fixed without community contributions. and removed needs: demo A CodePen demo or GitHub repository is needed to demonstrate the reproduction of the issue labels Mar 21, 2016
@topherfangio
Copy link
Contributor

@johnrivelt Thanks!

@EladBezalel Looks like it has a problem when the min value goes from larger to smaller.

@ThomasBurleson
Copy link
Contributor

This issue is closed as part of our ‘Surge Focus on Material 2' efforts.
For details, see our forum posting @ http://bit.ly/1UhZyWs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
P4: minor Minor issues. May not be fixed without community contributions.
Projects
None yet
Development

No branches or pull requests

4 participants