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

Button types should have Event parameter as value #1910

Closed
jlstevens opened this issue Jan 17, 2021 · 1 comment · Fixed by #1913
Closed

Button types should have Event parameter as value #1910

jlstevens opened this issue Jan 17, 2021 · 1 comment · Fixed by #1913
Labels
type: enhancement Minor feature or improvement to an existing feature

Comments

@jlstevens
Copy link
Contributor

Currently panel Buttons don't have a .value parameter like most other widgets. Instead, they have a clicks parameter that increments on each click.

With the addition of Event parameters in param, there is now a parameter that can represent transient events such as button clicks. The behavior is that the boolean value is True while callbacks are being processed post click and False otherwise. This lets you know which button was clicked if you have a callback that can be triggered by multiple different buttons.

I think this addition would help make panel buttons more consistent with other panel widgets and it would also help make buttons play nicely with other libraries (HoloViews in particular).

@jlstevens jlstevens added type: enhancement Minor feature or improvement to an existing feature TRIAGE Default label for untriaged issues labels Jan 17, 2021
@philippjfr philippjfr removed the TRIAGE Default label for untriaged issues label Jan 17, 2021
@jlstevens
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Minor feature or improvement to an existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants