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

Add change public method. #579

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

RomanBurunkov
Copy link

changes slider current 'from' and/or 'to' values, without recreating slider instance unlike the update method.

Change method allows to set from and to values without slider update, which redraws all dom.
Added is_changed flag which shows that change method is invoked.
Changed updateScene to drawHandlers for change method.
@RomanBurunkov
Copy link
Author

Hello @IonDen ,

Could you please take a look on this pull request.
I'm going to use that change method in my app but in a mean time have some doubts if it's ok and such way of changing from and to won't bring any issues with slider.

Thanks,
Roman.

Change method doesn't draw changes.
@IonDen
Copy link
Owner

IonDen commented Nov 27, 2018

Hi, Roman.
Very good idea, i will think about it.

@IonDen IonDen self-assigned this Nov 27, 2018
@IonDen IonDen added the Feature request Request for a new functionality label Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Request for a new functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants