Skip to content

Select type of event for slider widget#514

Merged
GuillaumeFavelier merged 2 commits intomasterfrom
slider_widget_event_type
Dec 23, 2019
Merged

Select type of event for slider widget#514
GuillaumeFavelier merged 2 commits intomasterfrom
slider_widget_event_type

Conversation

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor

This PR allows the user to specify how often the slider interacts with the callback.

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

Let me know what you think @banesullivan

Copy link
Copy Markdown
Member

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. It would be great to add real interactive testing, but at this point, it would take some serious work to simulate the callbacks in order to successfully trigger the correct events.

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

+1 for real interactive testing but I don't know where to start

@banesullivan
Copy link
Copy Markdown
Member

Interactive testing is definitely needed for the widgets, picking features, and more... perhaps down the road, we can implement a module for automatically interacting with all of these features

@banesullivan
Copy link
Copy Markdown
Member

This looks good to me!

@banesullivan
Copy link
Copy Markdown
Member

@GuillaumeFavelier, you should have full "maintainer" access now, so feel free to merge when you're ready

@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

Wow! Thank you so much! Okay I'll merge then

@GuillaumeFavelier GuillaumeFavelier merged commit 55032f8 into master Dec 23, 2019
@GuillaumeFavelier
Copy link
Copy Markdown
Contributor Author

Whoops, I forgot to edit the commit message before merging. I'll be careful from now on, sorry.

@banesullivan banesullivan deleted the slider_widget_event_type branch December 23, 2019 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Changes that enhance the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants