From e5a0f25d94034c6be45a4cc0b1c838df4f16c45e Mon Sep 17 00:00:00 2001
From: Nick O'Leary <nick.oleary@gmail.com>
Date: Thu, 9 Jul 2015 16:48:53 +0100
Subject: [PATCH] Keep sidebar tab menu in alphabetical order

---
 editor/js/ui/menu.js      | 44 ++++++++++++++++++++++++++++++---------
 editor/js/ui/sidebar.js   |  3 ++-
 locales/en-US/editor.json |  2 +-
 3 files changed, 37 insertions(+), 12 deletions(-)

diff --git a/editor/js/ui/menu.js b/editor/js/ui/menu.js
index 941e810d9..47fe824fc 100644
--- a/editor/js/ui/menu.js
+++ b/editor/js/ui/menu.js
@@ -29,7 +29,7 @@ RED.menu = (function() {
                 return null;
             }
         }
-        
+
         function setInitialState() {
             var savedStateActive = isSavedStateActive(opt.id);
             if (savedStateActive) {
@@ -52,12 +52,16 @@ RED.menu = (function() {
             item = $('<li class="divider"></li>');
         } else {
             item = $('<li></li>');
-            
+
+            if (opt.group) {
+                item.addClass("menu-group-"+opt.group);
+
+            }
             var linkContent = '<a '+(opt.id?'id="'+opt.id+'" ':'')+'tabindex="-1" href="#">';
             if (opt.toggle) {
                 linkContent += '<i class="fa fa-square pull-left"></i>';
                 linkContent += '<i class="fa fa-check-square pull-left"></i>';
-                
+
             }
             if (opt.icon !== undefined) {
                 if (/\.png/.test(opt.icon)) {
@@ -66,16 +70,16 @@ RED.menu = (function() {
                     linkContent += '<i class="'+(opt.icon?opt.icon:'" style="display: inline-block;"')+'"></i> ';
                 }
             }
-            
+
             if (opt.sublabel) {
                 linkContent += '<span class="menu-label-container"><span class="menu-label">'+opt.label+'</span>'+
                                '<span class="menu-sublabel">'+opt.sublabel+'</span></span>'
             } else {
                 linkContent += '<span class="menu-label">'+opt.label+'</span>'
             }
-            
+
             linkContent += '</a>';
-                
+
             var link = $(linkContent).appendTo(item);
 
             menuItems[opt.id] = opt;
@@ -139,7 +143,7 @@ RED.menu = (function() {
                     content: opt.tip
                 });
             }
-            
+
         }
 
 
@@ -154,8 +158,8 @@ RED.menu = (function() {
         //    $("#"+options.id+"-submenu").show();
         //    event.preventDefault();
         //});
-        
-        
+
+
         var topMenu = $("<ul/>",{id:options.id+"-submenu", class:"dropdown-menu pull-right"}).insertAfter(button);
 
         var lastAddedSeparator = false;
@@ -208,7 +212,27 @@ RED.menu = (function() {
     }
 
     function addItem(id,opt) {
-        createMenuItem(opt).appendTo("#"+id+"-submenu");
+        var item = createMenuItem(opt);
+        if (opt.group) {
+            var groupItems = $("#"+id+"-submenu").children(".menu-group-"+opt.group);
+            if (groupItems.length === 0) {
+                item.appendTo("#"+id+"-submenu");
+            } else {
+                for (var i=0;i<groupItems.length;i++) {
+                    var groupItem = groupItems[i];
+                    var label = $(groupItem).find(".menu-label").html();
+                    if (opt.label < label) {
+                        $(groupItem).before(item);
+                        break;
+                    }
+                }
+                if (i === groupItems.length) {
+                    item.appendTo("#"+id+"-submenu");
+                }
+            }
+        } else {
+            item.appendTo("#"+id+"-submenu");
+        }
     }
     function removeItem(id) {
         $("#"+id).parent().remove();
diff --git a/editor/js/ui/sidebar.js b/editor/js/ui/sidebar.js
index 8b855f2d2..b2f8ce6ff 100644
--- a/editor/js/ui/sidebar.js
+++ b/editor/js/ui/sidebar.js
@@ -64,7 +64,8 @@ RED.sidebar = (function() {
             label:options.name,
             onselect:function() {
                 showSidebar(options.id);
-            }
+            },
+            group: "sidebar-tabs"
         });
 
         knownTabs[options.id] = options;
diff --git a/locales/en-US/editor.json b/locales/en-US/editor.json
index e7b1f2c7d..3cb37c932 100644
--- a/locales/en-US/editor.json
+++ b/locales/en-US/editor.json
@@ -178,7 +178,7 @@
     },
     "sidebar": {
         "info": {
-            "name": "Node Information",
+            "name": "Information",
             "label": "info",
             "node": "Node",
             "type": "Type",