breadcrumb 1.6 behaviour in Projection

Topics: Administration, General
Nov 9, 2012 at 7:46 AM

When in a 'Projection' environment: I would NOT expect a 'breadcrumb' when clicking a 'Detail'. However, in 1.6 I get a full 'Menu list' as the breadcrumb!

? Do I need to get rid of the breadcrumb by the Placement.info (something like: <Place Parts_breadcrumb="-"/>) ?

See below details.
Thanks for your time and advise
ed

In 1.5.1 when clicking a Projection Summary item I do NOT get any 'breadcrumb' although the SourceCode shows (without list items!):
<div id="layout-before-main" class="group">
    <div class="zone zone-before-main"><article class="widget-breadcrumb widget-before-main widget-menu-widget widget">
</article></div>

HOWEVER, in 1.6:
Instead of 'a' breadcrumb I get the full Menu instead of 'a' breadcrumb (<ul class="menu menu-main-menu">).

<div id="layout-before-main" class="group">
    <div class="zone zone-before-main">
<article class="widget-breadcrumbmm widget-before-main widget-menu-widget widget">
  <nav>
    <ul class="menu menu-main-menu">
<li class="first">
    <a href="/">Home</a>
</li>
<li><a href="/pflanzen">Pflanzen</a>
</li>
<li>
    <a href="/slider">Slider</a>
</li>
<li class="dropdown">
    <a href="/parent">Parent</a>
  <ul>
<li>
    <a href="/child-10">Child 10</a>
</li>
        </ul>
</li>
<li class="last"><a href="/events">Events</a>
</li>
    </ul>
</nav>
</article></div>
</div>

Nov 13, 2012 at 2:09 PM

I think this is due to a bug in 1.6.  I have raised an issue http://orchard.codeplex.com/workitem/19246

Nov 13, 2012 at 2:14 PM

thanks for taking the time ... and raising an issue

ed

Nov 19, 2012 at 4:27 PM

applying your proposed fix  .....

The code is in MenuWidgetPartDriver.Display().

if (part.Breadcrumb && selectedPath != null) {
    menuItems = selectedPath;

Change to...
if (part.Breadcrumb) {
   menuItems = selectedPath ?? Enumerable.Empty<MenuItem>();

... solves part of the issue BUT shows on the Breadcrumb now the 'HOME' page which is not the 'correct' page.

Thanks for your effort

ed