Closed trafficonese closed 1 year ago
There is already a package on CRAN that provides access to this plugin.
Also, note that this repository (leaflet.extras) is currently not well maintained. I think @schloerke or someone else from RStudio currently only ensures that things here don't cause any trouble with leaflet releases, but there is no maintainence beyond that.
In case you are interested in taking the lead on maintaining it, please let us/me know. Ultimately, I think @jcheng5 should somehow alert to how RStudio wants to handle the maintainence void here, but having someone who is willing to maintain it would be a good start...
@tim-salabim Thanks, I saw this package when I finished implementing it here. The implementation differs a bit and I modified the JS source to give the user more options and to sync the slider with the buttons.
I find it easier to maintain the JS code if it is in a separate .js file rather than in the onRender
method, but that might just be a matter of taste. :)
Since I am just beginning to understand how all components (htmlWidgets, JS, R / Shiny) work together, I would be a little worried about being the maintainer of this package as some issues go beyond my understanding.
Nonetheless, if no one else is found and the R/Shiny/Leaflet gurus help me fix bugs / add new features, I would take on the maintainer role.
Is @bhaskarvk not maintaining it anymore?
No @bhaskarvk has left open source a long time ago unfortunately. He left me and the RStudio guys with admin rights for this package, but for me, same as you, most of the things here are beyond my skills, especially JS.
I sort of keep an eye on this repo as we use some parts from here in mapedit, but am too occupied with other package developments (mapview et al.) and don't want to take on leaflet.extras.
I suggest we wait a while and see if RStudio has an opinion on this.
I don’t think anyone on our side is signed up to maintain this package, sorry. That’s why this is not and has never been under the rstudio GitHub org.
@tim-salabim what parts are you using in mapview/mapedit?
@jcheng5 we use Leaflet.Draw plugin in mapedit.
I decided to transfer my open PRs with new features in a separate package leaflet.extras2. I will try to integrate the remaining feature requests there to make maintenance of this package a little easier.
Adds support for the plugin:
I changed and expanded the JS-code, so that it works without errors in R/Shiny. I also opened a PR at the original repo with those changes, but I'm not sure if it is still maintained since there are 2 other PR's waiting since 2015.
close #24