From 0a330544b377b337f12f2999dc59c21e0b59e167 Mon Sep 17 00:00:00 2001
From: Boris <boris.kvid@sitepointgroup.com>
Date: Fri, 21 Mar 2014 16:55:32 +1100
Subject: [PATCH] Prevent rendering of excessive <li> wrapper around a category
 breadcrumb which already is a <li>.

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.
---
 .../templates/components/bread-crumbs.js.handlebars       | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/app/assets/javascripts/discourse/templates/components/bread-crumbs.js.handlebars b/app/assets/javascripts/discourse/templates/components/bread-crumbs.js.handlebars
index ed285277f6c..902c4684860 100644
--- a/app/assets/javascripts/discourse/templates/components/bread-crumbs.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/components/bread-crumbs.js.handlebars
@@ -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>