-
Notifications
You must be signed in to change notification settings - Fork 75
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
Leaflet.extras addSearchFeatures not rendering #143
Comments
Hi I used one of your examples to add search feature code below. This runs and produces the map with the search box but when I type in New York (or any of the other city names) it location is not found. Do you have any ideas why this might be. Many thanks cities <- read.csv(textConnection("City,Lat,Long,Pop leaflet(cities) %>% addProviderTiles(providers$OpenStreetMap) %>% Hint! Search for ...
position='bottomright') |
I believe this is related to stefanocudini/leaflet-search#196. @schloerke I see that |
The circle marker issue looks to be fixed without testing. stefanocudini/leaflet-search#196 is not fixed as Path has not removed. Once Path is removed, I'll make a PR. |
Thanks for the update @schloerke! |
For what it's worth, @ramnathv I can confirm @schloerke 's comment. About two weeks ago I updated a local clone of leaflet.extras to the latest version of leaflet-search and still had to remove the Path bit to get it working properly. |
Thank you all for your comments, I'm a bit confused what I have to do to get this function to work. It clearly works in the Rpubs example. I am using R studio version 1.1.183, R version 3.4.2, leaflet version 2.0.1 and leaflet.extras version 1.0.0. I have tried to compare the html but it is very different. |
I am also having the same issue with all three addSearch options. addSearchUSCensusBureau and addSearchOSM result in the search icon being present on the leaflet map, however addSearchGoogle does not. Also puzzled by the fact that it is working great in the Rpubs example. I am running RStudio version: 1.1.456, R version: 3.4.4, leaflet version 2.0.1.9 and leaflet.extras version 1.0.0. Not sure what could be driving the issue, but wanted to contribute information on my system to help diagnose the problem. |
The feature you added has to have a label..
|
@tim-salabim, how to remove the Path bit to get it working properly? leaflet extra Search Button does not work for me either. |
This still appears to be a problem with leaflet 2.0.2 and leaflet.extras 1.0.0. It seems to be recognized that CircleMarkers don't work, but I haven't been able to get the RPubs addSearchFeatures() example to work with addCircles() either. |
I'll be resuming development of leaflet.extras in Dec and will tackle this in that round. Thanks! |
@bhaskarvk |
@bhaskarvk Hi, Will you work on the leaflet search button in Dec? Thank you |
I figured out how to make the search work with the CircleMarkers (removing the path check), you have to go into your R library path Open lfx-search-prod.js and search for "e instanceof t.Path ||" , and then delete it and save the file. Your CircleMarker search should work now |
@TacticalFate Are you editing the package files? Those directories don't exist for me. |
Thank you! Your solution works for me! |
I updated leaflet-search to v4.0.0 and |
allow searching of circle markers as per trafficonese#143
No description provided.
The text was updated successfully, but these errors were encountered: