-
Notifications
You must be signed in to change notification settings - Fork 640
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
Confusion around tileLayerOptions and layerOptions #625
Comments
+1 on this. Documentation and examples are unclear on when to use I, too, can only get |
This issue was moved to angular-ui/ui-leaflet#48 |
I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it. |
The use cases for
tileLayerOptions
andlayerOptions
are unclear in the examples and documentation.For example here:
angular-leaflet-directive/examples/custom-parameters-example.html
Lines 20 to 24 in 9cc9fd1
and here:
angular-leaflet-directive/test/unit/leafletDirectiveSpec.js
Lines 100 to 103 in dab1c1e
However in practice I have found that to get detectRetina to work, you need to put it inside
layerOptions
and nottileLayerOptions
as suggested.The two above links should be corrected, or the documentation clarified to remove confusion here.
The text was updated successfully, but these errors were encountered: