bhauman / lein-figwheel

Figwheel builds your ClojureScript code and hot loads it into the browser as you are coding!
Eclipse Public License 1.0
2.88k stars 209 forks source link

Notification doesn't always work with multiple garden builds and media queries #226

Open smahood opened 9 years ago

smahood commented 9 years ago

With the following use case, figwheel loads changes to the css properly but does not always show the figwheel notification.

When the CSS files are changed and reloaded, the notification doesn't always display - using the same example project, I've had it working every 2nd time, never, and every time. Not really predictable which will happen.

A sample project, as close to reproducible as I can get, can be found at https://github.com/smahood/figwheel-issue-226

bhauman commented 9 years ago

Thanks for the report. Figwheel hasn't been designed with any thought of multiple CSS file notifications.

This is a race condition that can be fixed with some judicious application of core.async. I'm already using for file loading extending this to css loading should be straight forward.