Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failed to parse SourceMap #732

Closed
raamdev opened this issue Apr 6, 2016 · 2 comments
Closed

Failed to parse SourceMap #732

raamdev opened this issue Apr 6, 2016 · 2 comments
Labels
Milestone

Comments

@raamdev
Copy link
Contributor

@raamdev raamdev commented Apr 6, 2016

The *.min.css files included with the distributable zip file currently contain a comment at the bottom that points to source maps (e.g., /*# sourceMappingURL=admin-bar.min.css.map */).

That comment is generally ignored by browsers (because it's just a comment). However, if your browser is configured in a way that causes it to look for and load the SourceMap files (again, for development purposes), then you'll see a (harmless) warning in the console:

2016-04-06_08-16-16

You might have a browser extension installed that is causing the warning to appear. More likely, your Google Chrome DevTools might be configured to look for and load SourceMaps:

2016-04-06_08-06-15

That said, there's no good reason for that comment to even appear there in the distributable versions of Comet Cache, since the source maps are only used for development purposes, so we should get that removed from future releases to avoid any confusion.

@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Apr 11, 2016

Next Release Changelog:

  • Bug Fix: Fixed an issue where some browsers would report "Failed to parse SourceMap" errors in their console when browsing the Comet Cache Options page. This was related to sourceMappingURL comments in the minified JS/CSS files that were intended for development purposes. Props to @1wdtv for reporting. See #732.
@raamdev raamdev closed this Apr 11, 2016
@raamdev
Copy link
Contributor Author

@raamdev raamdev commented Apr 16, 2016

Comet Cache v160416 has been released and includes changes from this GitHub Issue. See the v160416 announcement for further details.


This issue will now be locked to further updates. If you have something to add related to this GitHub Issue, please open a new GitHub Issue and reference this one (#732).

@wpsharks wpsharks locked and limited conversation to collaborators Apr 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant