{{ ASSET_PATH }} only seems to work in _includes/theme/themename/default.html, leaves off theme name in post.html and page.html about jekyll-bootstrap HOT 11 CLOSED

Same issue for me. Thanks for pointing out your workaround!

from jekyll-bootstrap.

Actually - here's how to do it. The default template explicitly declares the theme in the YAML front matter as:

theme :
name : design-co

Include that in your layout and you should be golden.

from jekyll-bootstrap.

I am having the same issue. I've placed a png in:

assets/themes/my-theme/images/foo.png

but {{ ASSET_PATH }} only prints out: `assets/themes//images/foo.png

I have the YAML front matter in the theme, I'm hoping to make my theme public so manually declaring the theme asset location in the _config.yml is not an option

from jekyll-bootstrap.

the issue appears to be in _includes/JB/setup

The var page.theme.name is empty, so the ASSET_PATH is not getting set with the theme name

from jekyll-bootstrap.

+1

from jekyll-bootstrap.

Still an issue... Just as @daithi-coombes described it.

from jekyll-bootstrap.

I created a patch.
https://github.com/fujii/jekyll-bootstrap/commit/16e8a9d10259e4665534825422ec05b704f7114d

from jekyll-bootstrap.

+1

from jekyll-bootstrap.

Housekeeping. I'll review the comment stream and @fujii's tweak. Will create PR after review / testing.

from jekyll-bootstrap.

OK. Discussion time. The fix from @fujii works as expected. There's one snag, though. And I don't think it's a show-stopper. Somewhat related to #227, but barely.

There's an issue with variable scope and ASSET_PATH. The fix will allow the themes to access assets using the variable, but actual posts, drafts, and pages won't have a valid ASSET_PATH. And I'm not 100% convinced that they need it.

Happy Path (for templates):

Gray Area (for posts, pages, and drafts):

If that's not an issue for you guys, I'm fine with just sticking a footnote somewhere that says that ASSET_PATH is only valid for themes. Pages, posts, and drafts are on their own.

from jekyll-bootstrap.

Tested without issue. No activity on comments. Merged.

from jekyll-bootstrap.