[gnome-shell/wip/aggregate-menu: 29/75] popupMenu: Ignore submenus when getting the column widths

[Jasper St. Pierre <jstpierre src gnome org>]

commit f67173929bfd5c6f7db3ded159b0fe6b8951cf96
Author: Jasper St. Pierre <jstpierre mecheye net>
Date:   Tue Jun 11 18:16:03 2013 -0400

    popupMenu: Ignore submenus when getting the column widths
    
    The new designs don't want these to be aligned the same way.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=702539

 js/ui/popupMenu.js |   10 ++++++++--
 1 files changed, 8 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/popupMenu.js b/js/ui/popupMenu.js
index a14abbd..c025b02 100644
--- a/js/ui/popupMenu.js
+++ b/js/ui/popupMenu.js
@@ -898,9 +898,12 @@ const PopupMenuBase = new Lang.Class({
         let columnWidths = [];
         let items = this.box.get_children();
         for (let i = 0; i < items.length; i++) {
-            if (!items[i].visible &&
-                !(items[i]._delegate instanceof PopupSubMenu && items[i-1].visible))
+            if (!items[i].visible)
                 continue;
+
+            if (items[i]._delegate instanceof PopupSubMenu)
+                continue;
+
             if (items[i]._delegate instanceof PopupBaseMenuItem || items[i]._delegate instanceof 
PopupMenuBase) {
                 let [itemMaxSpan, itemColumnWidths] = items[i]._delegate.getColumnWidths();
                 maxSpan = Math.max(maxSpan, itemMaxSpan);
@@ -916,6 +919,9 @@ const PopupMenuBase = new Lang.Class({
     setColumnWidths: function(widths) {
         let items = this.box.get_children();
         for (let i = 0; i < items.length; i++) {
+            if (items[i]._delegate instanceof PopupSubMenu)
+                continue;
+
             if (items[i]._delegate instanceof PopupBaseMenuItem || items[i]._delegate instanceof 
PopupMenuBase)
                 items[i]._delegate.setColumnWidths(widths);
         }