From 3f18d80fdf31396461883221a25e38e0609b2023 Mon Sep 17 00:00:00 2001 From: Ray Greenwell Date: Thu, 16 Apr 2009 21:11:03 +0000 Subject: [PATCH] Let's make all CommandMenus variableRowHeight. - What's the harm in it? Does it cause menu to do more calculation? What the fuck gus, they should have only implemented one code path and KISS. - it seems to be necessary when using separators. They are slim. When not used in a menu with separators, the separators are these hulking beasts. - it seems that when a submenu is added, things are going variable anyway. So: consistency. git-svn-id: svn+ssh://src.earth.threerings.net/nenya/trunk@809 ed5b42cb-e716-0410-a449-f6a68f950b19 --- src/as/com/threerings/flex/CommandMenu.as | 1 + 1 file changed, 1 insertion(+) diff --git a/src/as/com/threerings/flex/CommandMenu.as b/src/as/com/threerings/flex/CommandMenu.as index e038d6e1..0de5fea2 100644 --- a/src/as/com/threerings/flex/CommandMenu.as +++ b/src/as/com/threerings/flex/CommandMenu.as @@ -176,6 +176,7 @@ public class CommandMenu extends Menu itemRenderer = new ClassFactory(getItemRenderer()); verticalScrollPolicy = ScrollPolicy.OFF; + variableRowHeight = true; addEventListener(MenuEvent.ITEM_CLICK, itemClicked); }