Remove livereload cache buster param in extractId #2279
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When using grunt-contrib-watch for livereload and using less.js during development causes the reloading of less styles to happen incorrectly. Basically tiny-lr (used by grunt-contrib-watch) adds a query param to all the files it retrieves like "?livereload={{current time}}". Less.js then uses that url to try to extract the id of the given less file and sees each one as unique instead of the appropriate file.
This causes the livereload/less combination to add the styles from the changed less file but not remove the old ones. More discussion can be found here:
cgross/generator-cg-angular#60
Credit for researching the issue and determining the fix goes to @bpartridge.