From 5683ad9ddab216440373a0c7e6d9be021765b38f Mon Sep 17 00:00:00 2001
From: Petr Vobornik <pvoborni@redhat.com>
Date: Jan 21 2014 11:04:02 +0000
Subject: Change menu rendering to match RCUE structure


https://fedorahosted.org/freeipa/ticket/3902

---

diff --git a/install/ui/src/freeipa/widgets/Menu.js b/install/ui/src/freeipa/widgets/Menu.js
index 2aff779..2f78051 100644
--- a/install/ui/src/freeipa/widgets/Menu.js
+++ b/install/ui/src/freeipa/widgets/Menu.js
@@ -79,69 +79,138 @@ define(['dojo/_base/declare',
                 construct.empty(this.domNode);
             } else {
                 this.domNode = construct.create('div', {
-                    'class': 'navigation'
+                    'class': 'navbar primary persistent-secondary'
                 });
             }
+            this.innerNode = construct.create('div', {
+                'class': 'navbar-inner'
+            }, this.domNode);
             if (this.menu) {
-                this._render_children(null, this.domNode, 1);
+                this._render_children(null, this.innerNode, 1);
             }
             return this.domNode;
         },
 
         /**
-         * Render children of menu_item
-         * Top level items are rendered if menu_items is null
+         * Render submenu container of given level
          *
          * @protected
-         * @param {navigation.MenuItem|null} menu_item
-         * @param {HTMLElement} node
-         * @param {number} level
+         * @param {HTMLElement} node Node to add the container to.
+         * @param {Number} level submenu level
          */
-        _render_children: function (menu_item, node, level) {
-
-            var self = this;
-            var name = menu_item ? menu_item.name : null;
-            var children = this.menu.items.query({ parent: name },
-                                 { sort: [{attribute:'position'}]});
+        _render_level_container: function(node, level) {
 
             var lvl_class = this._get_lvl_class(level);
 
-            if (children.total > 0) {
-                var menu_node = construct.create('div', {
-                    'class': 'submenu ' + lvl_class
-                    //style: { display: 'none' }
-                });
+            var cont = construct.create('ul', {
+                'class': 'nav ' + lvl_class
+            }, node);
+            return cont;
+        },
 
-                if (menu_item) {
-                    attr.set(menu_node, 'data-item', menu_item.name);
-                }
+        /**
+         * Render item and submenu to container
+         */
+        _render_item: function(menu_item, container, level) {
 
-                var ul_node = construct.create('ul', null, menu_node);
+            var self = this;
+            var click_handler = function(event) {
+                self.item_clicked(menu_item, event);
+                event.preventDefault();
+            };
 
-                array.forEach(children, function(menu_item) {
+            var li_node = construct.create('li', {
+                'data-name': menu_item.name,
+                click: click_handler
+            }, container);
 
-                    var click_handler = function(event) {
-                        self.item_clicked(menu_item, event);
-                        event.preventDefault();
-                    };
+            var a_node = construct.create('a', {}, li_node);
+            this._update_item(menu_item, li_node);
 
-                    var li_node = construct.create('li', {
-                        'data-name': menu_item.name,
-                        click: click_handler
-                    }, ul_node);
+             // create submenu
+            this._render_children(menu_item, container, level + 1);
+        },
+
+        /**
+         * Render children of menu_item or top level items if menu_item is
+         * null.
+         *
+         * @protected
+         * @param {navigation.MenuItem|null} menu_item
+         * @param {HTMLElement} container
+         * @param {number} level
+         */
+        _render_children: function(menu_item, container, level) {
 
-                    var a_node = construct.create('a', {}, li_node);
+            var name = menu_item ? menu_item.name : null;
+            var children = this.menu.items.query({ parent: name },
+                                 { sort: [{attribute:'position'}]});
 
-                    this._update_item(menu_item, li_node);
+            if (children.total > 0) {
+                var item_container = this._render_level_container(container, level);
 
-                    // create submenu
-                    this._render_children(menu_item, menu_node, level + 1);
+                array.forEach(children, function(menu_item) {
+                    this._render_item(menu_item, item_container, level);
                 }, this);
 
-                construct.place(menu_node, node);
+                construct.place(item_container, container);
             }
         },
 
+//         /**
+//          * Render children of menu_item
+//          * Top level items are rendered if menu_items is null
+//          *
+//          * @protected
+//          * @param {navigation.MenuItem|null} menu_item
+//          * @param {HTMLElement} node
+//          * @param {number} level
+//          */
+//         _render_children2: function (menu_item, node, level) {
+//
+//             var self = this;
+//             var name = menu_item ? menu_item.name : null;
+//             var children = this.menu.items.query({ parent: name },
+//                                  { sort: [{attribute:'position'}]});
+//
+//             var lvl_class = this._get_lvl_class(level);
+//
+//             if (children.total > 0) {
+//                 var menu_node = construct.create('div', {
+//                     'class': 'submenu ' + lvl_class
+//                     //style: { display: 'none' }
+//                 });
+//
+//                 if (menu_item) {
+//                     attr.set(menu_node, 'data-item', menu_item.name);
+//                 }
+//
+//                 var ul_node = construct.create('ul', null, menu_node);
+//
+//                 array.forEach(children, function(menu_item) {
+//
+//                     var click_handler = function(event) {
+//                         self.item_clicked(menu_item, event);
+//                         event.preventDefault();
+//                     };
+//
+//                     var li_node = construct.create('li', {
+//                         'data-name': menu_item.name,
+//                         click: click_handler
+//                     }, ul_node);
+//
+//                     var a_node = construct.create('a', {}, li_node);
+//
+//                     this._update_item(menu_item, li_node);
+//
+//                     // create submenu
+//                     this._render_children(menu_item, menu_node, level + 1);
+//                 }, this);
+//
+//                 construct.place(menu_node, node);
+//             }
+//         },
+
         _get_lvl_class: function(level) {
             return this.level_class + '-' + level;
         },