{"action":"created","issue":{"url":"https://api.github.com/repos/rstudio/leaflet/issues/308","repository_url":"https://api.github.com/repos/rstudio/leaflet","labels_url":"https://api.github.com/repos/rstudio/leaflet/issues/308/labels{/name}","comments_url":"https://api.github.com/repos/rstudio/leaflet/issues/308/comments","events_url":"https://api.github.com/repos/rstudio/leaflet/issues/308/events","html_url":"https://github.com/rstudio/leaflet/issues/308","id":182753348,"number":308,"title":"addMouseCoordinates() no longer working","user":{"login":"jfroeschke","id":2806032,"avatar_url":"https://avatars.githubusercontent.com/u/2806032?v=3","gravatar_id":"","url":"https://api.github.com/users/jfroeschke","html_url":"https://github.com/jfroeschke","followers_url":"https://api.github.com/users/jfroeschke/followers","following_url":"https://api.github.com/users/jfroeschke/following{/other_user}","gists_url":"https://api.github.com/users/jfroeschke/gists{/gist_id}","starred_url":"https://api.github.com/users/jfroeschke/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jfroeschke/subscriptions","organizations_url":"https://api.github.com/users/jfroeschke/orgs","repos_url":"https://api.github.com/users/jfroeschke/repos","events_url":"https://api.github.com/users/jfroeschke/events{/privacy}","received_events_url":"https://api.github.com/users/jfroeschke/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-13T10:32:23Z","updated_at":"2016-10-13T17:49:37Z","closed_at":null,"body":"I'm working with leaflet in shiny using the mapview package to display coordinates of the mouse.  Previously it worked.  I upgraded the leaflet package and mapview to the latest versions and this function no longer works.  The example function:\r\n\r\nlibrary(mapview)\r\nleaflet() %>% addTiles() %>% addMouseCoordinates()\r\n\r\nR version 3.2.3 (2015-12-10)\r\nPlatform: x86_64-w64-mingw32/x64 (64-bit)\r\nRunning under: Windows 7 x64 (build 7601) Service Pack 1\r\n\r\n\r\nlocale:\r\n[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252   \r\n[3] LC_MONETARY=English_United States.1252 LC_NUMERIC=C                          \r\n[5] LC_TIME=English_United States.1252    \r\n\r\n\r\nattached base packages:\r\n[1] stats     graphics  grDevices utils     datasets  methods   base     \r\n\r\n\r\nother attached packages:\r\n[1] raster_2.5-8       sp_1.2-3           mapview_1.2.0      leaflet_1.0.2.9003\r\n\r\n\r\nloaded via a namespace (and not attached):\r\n [1] Rcpp_0.12.5         magrittr_1.5        gdalUtils_2.0.1.7   munsell_0.4.3      \r\n [5] viridisLite_0.1.3   colorspace_1.2-6    lattice_0.20-33     foreach_1.4.3      \r\n [9] plyr_1.8.4          tools_3.2.3         webshot_0.3.2       rgdal_1.1-10       \r\n[13] grid_3.2.3          R.oo_1.20.0         png_0.1-7           latticeExtra_0.6-28\r\n[17] htmltools_0.3.5     iterators_1.0.8     yaml_2.1.13         digest_0.6.9       \r\n[21] RColorBrewer_1.1-2  htmlwidgets_0.7     R.utils_2.3.0       codetools_0.2-14   \r\n[25] rsconnect_0.4.3     scales_0.4.0        R.methodsS3_1.7.1   stats4_3.2.3       \r\n[29] jsonlite_1.0        satellite_0.2.0\r\n\r\n  \r\n\r\ndoes not produce the expected result. Any ideas?"},"comment":{"url":"https://api.github.com/repos/rstudio/leaflet/issues/comments/253586783","html_url":"https://github.com/rstudio/leaflet/issues/308#issuecomment-253586783","issue_url":"https://api.github.com/repos/rstudio/leaflet/issues/308","id":253586783,"user":{"login":"tim-salabim","id":2826432,"avatar_url":"https://avatars.githubusercontent.com/u/2826432?v=3","gravatar_id":"","url":"https://api.github.com/users/tim-salabim","html_url":"https://github.com/tim-salabim","followers_url":"https://api.github.com/users/tim-salabim/followers","following_url":"https://api.github.com/users/tim-salabim/following{/other_user}","gists_url":"https://api.github.com/users/tim-salabim/gists{/gist_id}","starred_url":"https://api.github.com/users/tim-salabim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tim-salabim/subscriptions","organizations_url":"https://api.github.com/users/tim-salabim/orgs","repos_url":"https://api.github.com/users/tim-salabim/repos","events_url":"https://api.github.com/users/tim-salabim/events{/privacy}","received_events_url":"https://api.github.com/users/tim-salabim/received_events","type":"User","site_admin":false},"created_at":"2016-10-13T17:49:37Z","updated_at":"2016-10-13T17:49:37Z","body":"maybe it should, maybe not. I am not sure... The error stems from this part of `onRender`\r\n\r\n```\r\nmap.on('mousemove', function (e) {\r\n      lnlt.text(\", txt, \");\r\n    })\r\n```\r\nwhere map is earlier defined as `var map = this;` which worked with **leaflet_1.0.2.9000** but not anymore with **leaflet_1.0.2.9003**. So, even if it is a **mapview** function it seems to be related to the **leaflet** upgrade.\r\n\r\nAny thoughts, Bhaskar? "}}
{"actor":{"display_login":"tim-salabim"}}