]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Fix docs asset file paths in /grunt/configBridge.json (#20178)
authorChris Rebert <github@chrisrebert.com>
Sun, 26 Jun 2016 23:42:13 +0000 (16:42 -0700)
committerGitHub <noreply@github.com>
Sun, 26 Jun 2016 23:42:13 +0000 (16:42 -0700)
commitead5ed6d739ad2f5b773276b2ea4f34d4ea8d162
tree54ee4be03b6b09c8602985b886f9c5391ee03006
parent6cceeec695fdc5fb753703a7a55acd0ba9d62d92
Fix docs asset file paths in /grunt/configBridge.json (#20178)

Previously, when running the docs locally, the site, rooted at:
    http://localhost:9001/
would reference docs assets using relative URLs such as:
    /../assets/js/vendor/anchor.min.js
which is equivalent to:
    http://localhost:9001/../assets/js/vendor/anchor.min.js
which is nonsense, since the root directory has no parent directory.
Apparently browsers silently ignore this extra '..', hence why this wasn't noticed until now.
But if you adjust Jekyll's `baseurl` setting, this mistake causes incorrect URLs to get generated.

This commit corrects the problem by removing the extra '../' from the paths.

These paths are also referenced in the Gruntfile, where the fix actually allows us to simplify the code.
Previously, in the Gruntfile, we were doing, e.g.:
    path.join('./docs/assets', '../assets/js/vendor/anchor.min.js')
which calculates to:
    ./docs/assets/../assets/js/vendor/anchor.min.js
which can be simplified to:
    ./docs/assets/js/vendor/anchor.min.js
So we can remove the '/assets' suffix from the left argument
and the '../' prefix from the right argument
and still obtain the same result.
Gruntfile.js
grunt/configBridge.json