-
Notifications
You must be signed in to change notification settings - Fork 1
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 model
to htmlwidget
#28
Comments
I am not sure I understand the use case. Can you elaborate on the use case / what cannot be achieved without this change? I am not familiar with |
@keller-mark very fair question. I am still trying to wrap my head around all the different contexts/environments in which Also, I will caveat all of this discussion with I could very easily be missing something, and if so I am very sorry for my ignorance. So we produce the button widget as in the example in
Usually,
With the proposed change, we could do something like below.
How else might we achieve a similar result? Thanks so much for your consideration. |
I think this is fairly harmless and will not cause negative side effects. Could we add
model
to thehtmlwidget
so that it is accessible through theHTMLWidgets.find()
mechanism? See timelyportfolio@79804a5. Looks like the attachedmodel
will need to be aPromise
since render is async.Here is a quick example.
onRender
The text was updated successfully, but these errors were encountered: