Skip to content
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

wish: leaflet.wms GetFeatureInfo #84

Closed
Arevaju opened this issue May 25, 2017 · 11 comments
Closed

wish: leaflet.wms GetFeatureInfo #84

Arevaju opened this issue May 25, 2017 · 11 comments

Comments

@Arevaju
Copy link

Arevaju commented May 25, 2017

Plugin for Leaflet WMS GetFeatureInfo request: https://github.com/heigeo/leaflet.wms
Thanks

@bhaskarvk
Copy link
Contributor

The plugin only works under leaflet 1.x JS, while leaflet R package still uses 0.7.x. I will integrate the plugin as soon as leaflet moves to 1.x.

@rickyars
Copy link

Any clue when leaflet will move to 1.x?

@bhaskarvk
Copy link
Contributor

There's already some work in progress for that rstudio/leaflet#357

@bhaskarvk
Copy link
Contributor

Closing for now. Feel free to reopen when Leaflet goes to 1.x

@aTnT
Copy link

aTnT commented Apr 23, 2020

Has GetFeatureInfo been implemented? This seems to be a highly requested feature (see https://gis.stackexchange.com/questions/334004/enable-wms-getfeatureinfo-request-in-leaflet-using-r)

@trafficonese
Copy link
Owner

I started implementing this plugin in leaflet.extras2. You can find it here: https://github.com/trafficonese/leaflet.extras2.
It's not on CRAN yet, but I would be happy if you give it a try and tell me if it meets your needs.

@aTnT
Copy link

aTnT commented Apr 24, 2020

Thanks @trafficonese for the leaflet.extras2 package, this is great work! The GetFeatureInfo feature works well! The only small detail (probably unrelated to the leaflet.extras2::addWMS() function) is that i cannot control the popup box size (see picture below). I have tried unsuccessfully to change A and B within popupOptions(maxWidth = A, minWidth = B).

image

@trafficonese
Copy link
Owner

Thank you! I'm glad you find it useful!
I just pushed a commit, which includes the popupOptions argument for addWMS, so you should be able to include popupOptions = popupOptions(maxWidth = 600) for example.
I also included another example which does just that.

@aTnT
Copy link

aTnT commented Apr 24, 2020

Thanks again @trafficonese, it works now! :)

image

@MoritzMennenga
Copy link

MoritzMennenga commented Jun 15, 2020

I'd I tried around a lot, but I doenst't work in my case. I tried it with my data first and then I used your example https://github.com/trafficonese/leaflet.extras2/blob/master/inst/examples/wms_popup.R. If I use leaflet.extras2::addWMS() there is no WMS displayed. If I use the same with addWMSTiles (and without the popup) I can see the WMSmap but I can't use the popup Feature for the getFeatureInformation I really need.

@trafficonese
Copy link
Owner

Could you please open an issue here https://github.com/trafficonese/leaflet.extras2/issues and include your sessioninfo::session_info()? It works for me, but I am still on leaflet '2.0.1'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants