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.

Sign up for GitHub

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

Jump to bottom

Does mkdocs theme'sbootstrap.min.js need to contain sourceMappingURL comment? #3558

Open
hitsumabushi845 opened this issue Jan 23, 2024 · 1 comment
Open

Does mkdocs theme'sbootstrap.min.js need to contain sourceMappingURL comment? #3558

hitsumabushi845 opened this issue Jan 23, 2024 · 1 comment
Labels
Theme-mkdocs Issues specifically involving the mkdocs theme.

Comments

@hitsumabushi845
Copy link

MkDocs's mkdocs theme using bootstrap. Its bootstrap.min.js file contains sourceMappingURL comment.
But actually mkdocs theme doesn't contain bootstrap.min.js.map file in the theme directory.
Because of that, built website sometimes shows unwanted warning message on console.

Is it needed to contain?

mkdocs/mkdocs/themes/mkdocs/js/bootstrap.min.js

Line 7 in c591a67

//# sourceMappingURL=bootstrap.min.js.map

@tomchristie tomchristie added the Theme-mkdocs Issues specifically involving the mkdocs theme. label Apr 12, 2024
@tomchristie
Copy link
Member

We should either include the source map or remove the source map comment.

(I'd lean towards the latter.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Theme-mkdocs Issues specifically involving the mkdocs theme.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tomchristie @hitsumabushi845