Fredrik Blomqvist

I notice the `mousemove` event listener (that is attached to `document`) is not released in the `destroy()` method, causing a leak. /Fredrik Blomqvist

Fixes https://github.com/bevacqua/horsey/issues/69

change name of ".panel" class?

2

Maybe a silly request, but simple; could you consider renaming the ".panel" class to prefixed like ".layer-switcher-panel" or such? I happen to have a big pile of UI code that...

I think the SOLID principles would fit perfectly in a list like this! http://en.wikipedia.org/wiki/SOLID_(object-oriented_design) http://butunclebob.com/ArticleS.UncleBob.PrinciplesOfOod (personally I think the emphasis on OOD could be slightly reduced in the SOLID texts....

prevent double attachment

1

Hi, I think it would be useful to have a guard that would make sure only a single instance of the colorpicker is ever attached to an element. A simple...

I think "attribution" should be added to list of properties to be copied to the new source object. https://github.com/maphubs/mapbox-gl-arcgis-tiled-map-service/blob/957f5a0a4705e5e043532f77821b25d301941d7b/src/arcgis_tiled_map_service_source.js#L60

Not sure if intentional or not but the "assert" function is not baked into the dist source making it fail. Creating a global(window) level assert function makes it run again.

To make it build on Windows I had to relax the linebreak handling in eslint and prettier. `.eslint.json` Add rule: ``` "rules": { "linebreak-style": "off" } ``` Create a `.prettierrc`...