CSS refresh then reload about guard-livereload

[e2]

I think you can handle that with 2 rules:

guard 'livereload', {apply_css_live: true} do
    extensions = [:css, :js, :html, :png, :gif, :jpg, :jpeg]
    watch(%r{templates/.+\.(#{extensions * '|'})})
end

guard 'livereload', {apply_css_live: false} do
    extensions = [:js, :html, :png, :gif, :jpg, :jpeg]
    watch(%r{templates/.+\.(#{extensions * '|'})})
end

Does this help?

[brendon]

Thanks @e2, I tried that but it didn't work as expected. I then noticed that the regex was too permissive and was detecting a change to files like main.css.map. I put an end-of-string constraint into the regex and it works like a charm now with just the one rule:

guard 'livereload', apply_css_live: true do
    extensions = [:css, :js, :html, :png, :gif, :jpg, :jpeg]
    watch(%r{templates/.+\.(#{extensions * '|'})\z})
end

Feel free to close this if unless there's something you'd like to add :)

[e2]

Thanks! I added a test, fixed the template and merged to master.

[brendon]

You're welcome :) Glad to be of some help. I assume the \z is the better approach vs $? Is it only ever receiving one path at a time?

[e2]

Yes, the watch is matched against all files individually (out of the batch of changed files).

The "\z" and "$" make no difference, unless people have newlines in their file names (unlikely).

If you have any suggestions, feel free to open issues with ideas. I'm sometimes too deep into these projects, so it's easy for me to miss the obvious.

Tweaks like these seem small, but they actually save people a lot of time and frustration - and that's a big thing.

[brendon]

Thanks @e2, that makes sense to me. My template developer is much happier with the performance of not having to refresh the whole page now :)

Thanks also for your feedback. It's great to hear these things make a difference for others :)