IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-12-01 04:59:22 -06:00
Code Issues Packages Projects Releases Wiki Activity

Prevent rendering of excessive <li> wrapper around a category breadcrumb which already is a <li>.

Browse Source 
The excessive <li> wrapper coming from the template is automatically closed by the browser during the HTML parsing step, causing an empty <li> being visible in the rendered source (preceding the first breadcrumb). This causes a visual bug when not using the default Discourse stylesheet, because the extra LI element pushes the content to the right and sometimes also renders an extra black pixel.
This commit is contained in:
Boris 2014-03-21 16:55:32 +11:00
parent 2dc5e3c81e
commit 0a330544b3
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
app/assets/javascripts/discourse/templates/components/bread-crumbs.js.handlebars
View File

@ -1,11 +1,7 @@
<li>
{{category-drop category=firstCategory categories=parentCategories}}
</li>
{{category-drop category=firstCategory categories=parentCategories}}
{{#if childCategories}}
<li>
{{category-drop category=secondCategory parentCategory=firstCategory categories=childCategories subCategory="true" noSubcategories=noSubcategories}}
</li>
{{category-drop category=secondCategory parentCategory=firstCategory categories=childCategories subCategory="true" noSubcategories=noSubcategories}}
{{/if}}
<div class='clear'></div>