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

404 on a source map on getbootstrap.com #10839

Closed
mgol opened this issue Sep 27, 2013 · 4 comments
Closed

404 on a source map on getbootstrap.com #10839

mgol opened this issue Sep 27, 2013 · 4 comments

Comments

@mgol
Copy link

mgol commented Sep 27, 2013

This is an issue with the site getbootstrap.com, not the framework but I don't see any other place to post it.

The file: http://getbootstrap.com/assets/js/jquery.js has the URL to the source map file included but there is no such file in your assets which causes 404 when one has source map support enabled and an open console.

Removing the second line:

//@ sourceMappingURL=jquery-1.10.2.min.map

solves the issue.

@huwcbjones
Copy link

huwcbjones commented Sep 28, 2013

The jquery map is used for debugging jquery. It is optional whether you choose to include it or not - it just makes debugging easier :-)
Source: http://jquery.com/download/

@mgol
Copy link
Author

mgol commented Sep 29, 2013

@hugebones I'm very well aware what's the purpose of source maps. However, if you choose to not include the map file in your assets, you should remove the source map declaration from the minified file so that it doesn't generate excessive HTTP requests to the site when one opens a console. This one line is just not needed anymore - it adds bytes and results in errors. That's true they're not critical but they're there.

TBH for me it's the issue with source map specification though avoidable in this case.

@zlatanvasovic
Copy link
Contributor

@mzgol You are right. This should be removed.

@cvrebert
Copy link
Collaborator

This is no longer an issue due to #11013.

racke added a commit to interchange/TableEditor that referenced this issue Mar 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants