-
Notifications
You must be signed in to change notification settings - Fork 206
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
make widgets also available as a web asset #1
Comments
So I made some changes in my fork with this commit. While I do not think this is the ideal way to handle, I think something like this is much better.
Then this source the R code and the assets are defined as web resources instead of local files. The really ugly part of this is the
I guess we could in theory do both with |
I think the decision on this is to require an R package to make a widget which will copy the appropriate dependencies into a local directory. While I still would love to see |
For what it is worth, with minimal tweaks, I was able to make widgets work in a non-package setting. The idea is for widgets to be contained in a folder that follows some conventions.
In the |
That would be very nice, but thinking more in the context of cdn for widgets where say a gh-pages branch can serve up a widget, so nothing needs to be stored or copied locally. I'm sure you could make it happen but also know we'll run into pandoc issues here and violate the multi-context principle. |
If
htmlwidgets
is as successful as I think it will be, a separate package for each widget will get quite unwieldy. Maybe think about each widget as both an R package or web-available structure (gh-pages branch), so that a separate install is not required. I thinkhtmlwidgets
really is the only thing that absolutely needs to be a package.Think about all the
install_github
issues we have had withrCharts
andslidify
and this very quickly becomes nightmarish. If we could gethtmlwidgets
on CRAN, and it should be simple since it is so targeted, then things become much easier if there is only one package install.The text was updated successfully, but these errors were encountered: