namespace providers to allow addMiniMap to be more accessible (#568)

* to make it accessible within other packages (see https://github.com/mtennekes/tmap/issues/231) * add minimap fix to NEWS * update news item
rstudio · Nov 1, 2018 · 419e07d · 419e07d
1 parent 145eb8c
commit 419e07d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/NEWS b/NEWS
@@ -13,6 +13,7 @@ leaflet 2.0.2
 BUG FIXES and IMPROVEMENTS
 * Require viridis >= 0.5.1 to avoid namespace issues with viridisLite (#557)
 * Fixed broken mouse events after using leaflet-search from leaflet.extras within shiny applications (#563)
+* Use leaflet namespace on `providers` in `addMiniMap` to make the function accessible in other packages. Fixes [mtennekes/tmap#231](https://github.com/mtennekes/tmap/issues/231). (#568)
 * Require scales >= 1.0.0 to avoid exact color matching issues (#578)
 
 

diff --git a/R/plugin-minimap.R b/R/plugin-minimap.R
@@ -88,7 +88,7 @@ addMiniMap <- function(
   tilesURL <- NULL
   tilesProvider <- NULL
  if (!is.null(tiles)) {
-   if (tiles %in% providers) {
+   if (tiles %in% leaflet::providers) {
       map$dependencies <- c(map$dependencies, leafletProviderDependencies())
       tilesProvider <- tiles
     } else {