vuepress / vuepress-theme-blog

Default blog theme for VuePress.
https://vuepress-theme-blog.billyyyyy3320.com/
MIT License
209 stars 127 forks source link

Fix base url in header #9

Closed burmecia closed 5 years ago

burmecia commented 5 years ago

Summary

When the blog is placed in a sub-folder, base url is already presented. So don't need to append the base url again, otherwise it will be duplicated.

For example, when place the blog inside another app's folder blog, we will set the base.url to /blog/ in .vuepress/config.js. Thus, the header url will become /blog/blog which is incorrect.

What kind of change does this PR introduce? (check at least one)

If changing the UI of default theme, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

You have tested in the following browsers: (Providing a detailed version will be better.)

If adding a new feature, the PR's description includes:

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information: