From a41bfcbfafc157f475cba65c73540ab427229508 Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Tue, 25 Feb 2014 10:13:36 +0000
Subject: [PATCH] Restructure outer layout

---
 application/layouts/scripts/body.phtml        | 22 ++++++++++++---
 application/layouts/scripts/parts/menu.phtml  |  2 +-
 .../layouts/scripts/parts/navigation.phtml    | 27 ++++++++++---------
 3 files changed, 34 insertions(+), 17 deletions(-)

diff --git a/application/layouts/scripts/body.phtml b/application/layouts/scripts/body.phtml
index 41c275d69..995d3baae 100755
--- a/application/layouts/scripts/body.phtml
+++ b/application/layouts/scripts/body.phtml
@@ -1,9 +1,26 @@
 <?php if (Icinga\Authentication\Manager::getInstance()->isAuthenticated()): ?>
 <div id="layout">
  <div id="header">
- <ul id="notifications"></ul>
+   <ul id="notifications"></ul>
+<div id="logo"><a href="<?= $this->href('/dashboard') ?>"><img src="<?= $this->href('img/logo_icinga-inv.png') ?>" class="logo" /></a>
+</div>
  </div>
- <div id="inner-layout">
+<!--
+  <ul class="tabs" >
+    <li class="dropdown">
+      <a href="#" class="dropdown-toggle">
+      <?= $this->img('icons/user.png') ?>
+      <?= $this->escape($this->auth()->getUser()->getUsername()) ?>
+        <b class="caret"></b>
+      </a>
+      <ul class="dropdown-menu">
+        <li><a href="<?= $this->href('/preference') ?>"><?= $this->translate('Preferences'); ?></a></li>
+        <li><a href="<?= $this->href('/authentication/logout'); ?>" title="Logout"><?= $this->translate('Logout'); ?></a></li>
+      </ul>
+    </li>
+  </ul>
+-->
+
  <div id="sidebar">
   <?= $this->layout()->navigation ?>
   <?php echo $this->render('parts/navigation.phtml') ?>
@@ -18,7 +35,6 @@
   <div id="col3" class="container">
   </div>
  </div><!-- END of main -->
-</div><!-- END of inner-layout -->
 </div><!-- END of layout -->
 <?php else: ?>
 <?= $this->render('inline.phtml') ?>
diff --git a/application/layouts/scripts/parts/menu.phtml b/application/layouts/scripts/parts/menu.phtml
index 7955f3251..b07cc641a 100644
--- a/application/layouts/scripts/parts/menu.phtml
+++ b/application/layouts/scripts/parts/menu.phtml
@@ -11,7 +11,7 @@ foreach ($this->items as $item) {
             $item->getIcon(),
             array('height' => '16px', 'width' => '16px')
         ) . ' ' : '',
-        $item->getTitle()
+        $this->escape($item->getTitle())
     );
 
     if ($item->hasChildren()) {
diff --git a/application/layouts/scripts/parts/navigation.phtml b/application/layouts/scripts/parts/navigation.phtml
index f4b27a828..99c8e07c4 100755
--- a/application/layouts/scripts/parts/navigation.phtml
+++ b/application/layouts/scripts/parts/navigation.phtml
@@ -1,19 +1,20 @@
-<div id="menu-header"><img src="<?= $this->href('img/logo_icinga-inv.png') ?>" style="height: 2em; width:6em; float: left; margin-left: 1.5em; margin-right: 9em; margin-top: 0.5em;" alt="" /></div>
-<div id="menu">
-<input type="text" style="width: 9em; margin: 0.5em;background-color: #999; background-image: url('<?= $this->href('img/icons/search.png') ?>'); background-repeat: no-repeat; background-position: 0.8em center; padding: 0.3em; padding-left: 2.5em; border: none; color: #F5F5F5; text-shadow: -1px 1px 0 #555555;" placeholder="Search..." />
 <?php
-// determine current key
+
+// Don't render a menu for unauthenticated users unless menu is auth aware
+if (! $this->auth()->isAuthenticated()) {
+    return;
+}
+
+// Current url
 $url = Icinga\Web\Url::fromRequest()->getRelativeUrl();
 $menu = Icinga\Web\Menu::fromConfig();
 
-if ($this->auth()->isAuthenticated()) {
-    echo $this->partial(
-       'parts/menu.phtml',
-        array(
-            'items' => $menu->getChildren(),
-            'url'   => $url
-        )
-    );
-}
+?>
+<div id="menu">
+  <form action="<?= $this->href('search') ?>" method="post"><input type="text" name="q" class="search" placeholder="Search..." autocomplete="off" autocorrect="off" autocapitalize="off" spellcheck="false" /></form>
+<?= $this->partial('parts/menu.phtml', array(
+    'items' => $menu->getChildren(),
+    'url'   => $url
+))
 ?>
 </div>