-
Notifications
You must be signed in to change notification settings - Fork 112
feat: Improvements to selectize plugin defaults/updates #2064
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
base: main
Are you sure you want to change the base?
Conversation
64694e8
to
b2e5799
Compare
b2e5799
to
e350db0
Compare
b86a561
to
a720e43
Compare
selectize_deps(), | ||
type="application/json", | ||
data_for=resolved_id, | ||
# Which option values should be interpreted as JS? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you saying the next line addresses this, or is this an open question?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The former
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I might make that clearer? But up to you
This PR pairs with rstudio/shiny#4276 (and, less importantly, rstudio/shiny#4275) to do the following:
Fixes Inconsistent behavior when specifying options in update_selectize() #2055
update_selectize(options=...)
now preserves it's default plugins (i.e.,remove_button
/clear_button
) correctly (in both theserver=True/False
cases).Adds a
remove_button
parameter toupdate_selectize()
so that the user can easily remove/add if that behavior should change from some reason.Adds support for a
input_selectize(remove_button="both")
value, which can be useful withmultiple=True
TODO
clear_button
fix: improve styling of 'clear_button' selectize plugin rstudio/shiny#4277shinyRemoveButton
selectize.js option (for py-shiny) rstudio/shiny#4276 and fix: add CSS workaround to hide all but the first selectize remove button rstudio/shiny#4275))htmlDependencies.R
to point atmain