modules/gallery/views/menu.html.php

<?php defined("SYSPATH") or die("No direct script access.") ?>
<? if ($menu->elements): // Don't show the menu if it has no choices ?>
<? if ($menu->is_root): ?>
<ul <?= isset($menu->css_id) ? "id='$menu->css_id'" : "" ?> class="<?= $menu->css_class ?>">
  <? foreach ($menu->elements as $element): ?>
  <?= $element->render() ?>
  <? endforeach ?>
</ul>

<? else: ?>

<li title="<?= $menu->label->for_html_attr() ?>">
  <a href="#">
    <?= $menu->label->for_html() ?>
  </a>
  <ul>
    <? foreach ($menu->elements as $element): ?>
    <?= $element->render() ?>
    <? endforeach ?>
  </ul>
</li>

<? endif ?>
<? endif ?>
Change our menu building blocks to use PHP templates so that themes can override them and define their own menu formats. I worry a little bit that this approach may be too heavy since we're now doing a lot more template includes than we were before. Also, I had to change the Menu API to stop using __toString() because you can't throw exceptions from __toString() which would make it an unhappy experience for developers. 2009-10-27 13:46:24 -07:00			`<?php defined("SYSPATH") or die("No direct script access.") ?>`
Remove Menu::compact() in favor of putting an if-then clause in menu.html.php. This serves two purposes: 1) It's more efficient since we're doing less passes over the Menu tree 2) We're allowing themers to decide whether or not to show empty menus 2009-10-27 14:00:32 -07:00			`<? if ($menu->elements): // Don't show the menu if it has no choices ?>`
Change our menu building blocks to use PHP templates so that themes can override them and define their own menu formats. I worry a little bit that this approach may be too heavy since we're now doing a lot more template includes than we were before. Also, I had to change the Menu API to stop using __toString() because you can't throw exceptions from __toString() which would make it an unhappy experience for developers. 2009-10-27 13:46:24 -07:00			`<? if ($menu->is_root): ?>`
Modify the Menu_Element class to allow for the specification of a custom view. Also allow the root element to specify a css id. 2009-11-06 13:03:48 -08:00			`<ul <?= isset($menu->css_id) ? "id='$menu->css_id'" : "" ?> class="<?= $menu->css_class ?>">`
Oops. Change "$this" to "$menu" in our menu component views. 2009-10-27 13:57:36 -07:00			`<? foreach ($menu->elements as $element): ?>`
Change our menu building blocks to use PHP templates so that themes can override them and define their own menu formats. I worry a little bit that this approach may be too heavy since we're now doing a lot more template includes than we were before. Also, I had to change the Menu API to stop using __toString() because you can't throw exceptions from __toString() which would make it an unhappy experience for developers. 2009-10-27 13:46:24 -07:00			`<?= $element->render() ?>`
			`<? endforeach ?>`
			`</ul>`

			`<? else: ?>`

			`<li title="<?= $menu->label->for_html_attr() ?>">`
			`<a href="#">`
			`<?= $menu->label->for_html() ?>`
			`</a>`
			`<ul>`
Oops. Change "$this" to "$menu" in our menu component views. 2009-10-27 13:57:36 -07:00			`<? foreach ($menu->elements as $element): ?>`
Change our menu building blocks to use PHP templates so that themes can override them and define their own menu formats. I worry a little bit that this approach may be too heavy since we're now doing a lot more template includes than we were before. Also, I had to change the Menu API to stop using __toString() because you can't throw exceptions from __toString() which would make it an unhappy experience for developers. 2009-10-27 13:46:24 -07:00			`<?= $element->render() ?>`
			`<? endforeach ?>`
			`</ul>`
			`</li>`

			`<? endif ?>`
Remove Menu::compact() in favor of putting an if-then clause in menu.html.php. This serves two purposes: 1) It's more efficient since we're doing less passes over the Menu tree 2) We're allowing themers to decide whether or not to show empty menus 2009-10-27 14:00:32 -07:00			`<? endif ?>`