search

pattex / jekyll-tagging

Jekyll plugin to automatically generate a tag cloud and tag pages.
385 stars 76 forks source link

Slugs not compatible with Liquid #54

Open akerbos opened 8 years ago

akerbos commented 8 years ago

When creating links to tag pages, something like this is intuitive:

{% for tag in page.tags %}
  <li><a href="{{ site.baseurl }}/{{ site.tag_page_dir }}/{{ tag | slugify }}.html">{{ tag }}</a></li>
{% endfor %}

Unfortunately, slugify is not compatible to your jekyll_tagging_slug.

I don't know how robust jekyll_tagging_slug is but you may want to switch to using slugify for consistency with other Jekyll stuff.

akerbos commented 8 years ago

FWIW, {{ tag | jekyll_tagging_slug }} works and does create the correct URLs. If this request gets declined, please document this. Thanks!