From b1ee36e3aa593884c208d005aac21bd840221908 Mon Sep 17 00:00:00 2001
From: martimiz <martine@balbus.nl>
Date: Wed, 8 Aug 2012 17:30:20 +0200
Subject: [PATCH] Fix: display the correct (menu) icon in the
 GridFieldDetailForm's breadcrumbs.

In CMSBreadcrumbs.ss the (TopLevel)Controller is not recognized within
the Breadcrumbs loop, so the class that is used in css to style the
icon cannot be created. Moving the construction outside the loop will
remedy this. As far as I can see this works within the CMS.
---
 admin/templates/CMSBreadcrumbs.ss | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/admin/templates/CMSBreadcrumbs.ss b/admin/templates/CMSBreadcrumbs.ss
index 92ad8ea6a..e5ddd23bd 100644
--- a/admin/templates/CMSBreadcrumbs.ss
+++ b/admin/templates/CMSBreadcrumbs.ss
@@ -1,22 +1,19 @@
 <div class="breadcrumbs-wrapper" data-pjax-fragment="Breadcrumbs">
-	<% loop Breadcrumbs %>
-	
-		<% if First %>
-			<% if ToplevelController %>
-				<span class="section-icon icon icon-16 icon-{$ToplevelController.MenuCurrentItem.Code.LowerCase}"></span>
-			<% else_if Controller %>
-				<span class="section-icon icon icon-16 icon-{$Controller.MenuCurrentItem.Code.LowerCase}"></span>
-			<% else %>
-				<span class="section-icon icon icon-16 icon-{$MenuCurrentItem.Code.LowerCase}"></span>
-			<% end_if %>
-		<% end_if %>
 
+	<% if ToplevelController %>
+		<span class="section-icon icon icon-16 icon-{$ToplevelController.MenuCurrentItem.Code.LowerCase}"></span>
+	<% else_if Controller %>
+		<span class="section-icon icon icon-16 icon-{$Controller.MenuCurrentItem.Code.LowerCase}"></span>
+	<% else %>
+		<span class="section-icon icon icon-16 icon-{$MenuCurrentItem.Code.LowerCase}"></span>
+	<% end_if %>
+
+	<% loop Breadcrumbs %>
 		<% if Last %>
 			<span class="cms-panel-link crumb last">$Title.XML</span>
 		<% else %>
 			<a class="cms-panel-link crumb" href="$Link">$Title.XML</a>
 			<span class="sep">/</span>
 		<% end_if %>
-
 	<% end_loop %>
 </div>