search

Leaflet / Leaflet.fullscreen

A fullscreen control for Leaflet
http://leaflet.github.io/Leaflet.fullscreen/
ISC License
436 stars 117 forks source link

Control icon is not a full screen icon #66

Closed completer closed 7 years ago

completer commented 7 years ago

Hello, I don't think I've ever seen a full screen icon that looks like the current pair of icons used for this control (which look more like a target or selector).

What would be the way to change it to something that made more sense to users? For example, to: http://fontawesome.io/3.2.1/icon/fullscreen/

IvanSanchez commented 7 years ago

Well, there are precendents of icons like the one used, see e.g. https://materialdesignicons.com/icon/fullscreen

The way to override the icon is to override this CSS rule. If you can come up with another way of specifying the icon, feel free to suggest it in the form of a pull request.

seyfro commented 7 years ago

FYI - I made 2 alternative icon sets for my plugin www.mapsmarker.com, feel free to use it (just use the proper credits and license, which is P.J. Onori, http://somerandomdude.com, Creative Commons Attribution Share Alike (by-sa)

fullscreen 1x fullscreen 2x

fullscreen 1x-light fullscreen 2x-light