search

plusjade / jekyll-bootstrap

The quickest way to start and publish your Jekyll powered blog. 100% compatible with GitHub pages.
jekyllbootstrap.com
MIT License
3.35k stars 1.71k forks source link

the resource path is wrong #320

Open jokerkiller opened 7 years ago

jokerkiller commented 7 years ago

the request path is http://localhost:4000/assets/themes/bootstrap-3/bootstrap/css/bootstrap.min.css ,but the real path is assets/themes/bootstrap/bootstrap/css/bootstrap.min.css. please change

ghost commented 7 years ago

Broken at https://github.com/plusjade/jekyll-bootstrap/commit/1d0f15edaa537923111240b86e88ce185db97636

trentmillar commented 7 years ago

change _includes/JB/setup

from

{% capture ASSET_PATH %}{{ BASE_PATH }}/assets/themes/{{ page.theme.name }}{% endcapture %}

to

{% capture ASSET_PATH %}{{ BASE_PATH }}/assets/themes/{% endcapture %}

Not sure if this is the right approach but it works. Seems like this project is forgotten?

trentmillar commented 7 years ago

I installed a theme and it expects

{% capture ASSET_PATH %}{{ BASE_PATH }}/assets/themes/{{ layout.theme.name }}{% endcapture %}

Just expect to do a tweak or two to get it running.

ryanmonro commented 7 years ago

I moved /assets/themes/bootstrap into a new dir /assets/themes/bootstrap-3/ as a temporary fix to get the Bootstrap theme working. The Twitter theme still seems to work fine though.