svn commit: r14578 - trunk/src/argouml-app/src/org/argouml/ui/cmd

Author: tfmorris
Date: 2008-05-01 15:34:20-0700
New Revision: 14578

Modified:
   trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java
   trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java

Log:
TODOs

Modified: trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java?view=diff&rev=14578&p1=trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java&p2=trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java&r1=14577&r2=14578
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/ui/cmd/GenericArgoMenuBar.java	2008-05-01 15:34:20-0700
@@ -919,6 +919,8 @@
      * Build the menu "Critique".
      */
     private void initMenuCritique() {
+        // TODO: This creates a dependency on the Critics subsystem.
+        // Instead that subsystem should register its desired menus and actions.
         critique =
             (ArgoJMenu) add(new ArgoJMenu(MENU + prepareKey("Critique")));
         setMnemonic(critique, "Critique");

Modified: trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java?view=diff&rev=14578&p1=trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java&p2=trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java&r1=14577&r2=14578
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/ui/cmd/ShortcutMgr.java	2008-05-01 15:34:20-0700
@@ -660,6 +660,8 @@
                 KeyEvent.VK_F7, 0), new ActionGenerateAll());
 
         // critique menu
+        // TODO: This dependency should be inverted with the Critics subsystem
+        // registering its desired shortcuts with us - tfm
         putDefaultShortcut(ACTION_AUTO_CRITIQUE, null, 
                 new ActionAutoCritique());
         putDefaultShortcut(ACTION_OPEN_DECISIONS, null,
@@ -667,7 +669,7 @@
         putDefaultShortcut(ACTION_OPEN_GOALS, null, new ActionOpenGoals());
         putDefaultShortcut(ACTION_OPEN_CRITICS, null, new ActionOpenCritics());
 
-        // critique menu
+        // help menu
         putDefaultShortcut(ACTION_SYSTEM_INFORMATION, null,
                 new ActionSystemInfo());
         putDefaultShortcut(ACTION_ABOUT_ARGOUML, null,