X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fgui%2FViewSelectionMenu.java;h=f1e246706775639613d327542106b858b84bc2d2;hb=f2d38aaebf121e49be29cb19ae01f1c6a36fe7e0;hp=e9cdcef56013d01b9317ed88114fed1d8a78d9a3;hpb=47168f025aefdaa044802bd5f8f510ffe43a4808;p=jalview.git

diff --git a/src/jalview/gui/ViewSelectionMenu.java b/src/jalview/gui/ViewSelectionMenu.java
index e9cdcef..f1e2467 100644
--- a/src/jalview/gui/ViewSelectionMenu.java
+++ b/src/jalview/gui/ViewSelectionMenu.java
@@ -1,6 +1,6 @@
 /*
- * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8.2)
- * Copyright (C) 2014 The Jalview Authors
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
  * 
  * This file is part of Jalview.
  * 
@@ -27,8 +27,8 @@ import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
 import java.awt.event.ItemEvent;
 import java.awt.event.ItemListener;
+import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
-import java.awt.event.MouseListener;
 import java.util.List;
 
 import javax.swing.JCheckBoxMenuItem;
@@ -150,9 +150,10 @@ public class ViewSelectionMenu extends JMenu
     {
       // ensure we update menu state to reflect external selection list state
       append = append || _selectedviews.size() > 1;
-      toggleview = new JCheckBoxMenuItem("Select many views", append);
-      toggleview
-              .setToolTipText(MessageManager.getString("label.toggle_enabled_views"));
+      toggleview = new JCheckBoxMenuItem(
+              MessageManager.getString("label.select_many_views"), append);
+      toggleview.setToolTipText(MessageManager
+              .getString("label.toggle_enabled_views"));
       toggleview.addItemListener(new ItemListener()
       {
 
@@ -170,7 +171,8 @@ public class ViewSelectionMenu extends JMenu
 
       });
       add(toggleview);
-      add(selectAll = new JMenuItem(MessageManager.getString("label.select_all_views")));
+      add(selectAll = new JMenuItem(
+              MessageManager.getString("label.select_all_views")));
       selectAll.addActionListener(new ActionListener()
       {
 
@@ -192,7 +194,8 @@ public class ViewSelectionMenu extends JMenu
           }
         }
       });
-      add(invertSel = new JMenuItem(MessageManager.getString("label.invert_selection")));
+      add(invertSel = new JMenuItem(
+              MessageManager.getString("label.invert_selection")));
       invertSel.addActionListener(new ActionListener()
       {
 
@@ -214,15 +217,14 @@ public class ViewSelectionMenu extends JMenu
       invertSel.setEnabled(append);
       selectAll.setEnabled(append);
     }
-    for (AlignmentPanel ap : allviews)
+    for (final AlignmentPanel ap : allviews)
     {
       String nm = ((ap.getViewName() == null || ap.getViewName().length() == 0) ? ""
               : ap.getViewName() + " for ")
               + ap.alignFrame.getTitle();
-      final JCheckBoxMenuItem a = new JCheckBoxMenuItem(nm,
+      final JCheckBoxMenuItem checkBox = new JCheckBoxMenuItem(nm,
               _selectedviews.contains(ap));
-      final AlignmentPanel p = ap;
-      a.addItemListener(new ItemListener()
+      checkBox.addItemListener(new ItemListener()
       {
         @Override
         public void itemStateChanged(ItemEvent e)
@@ -233,15 +235,15 @@ public class ViewSelectionMenu extends JMenu
             {
               enabled = false;
               // toggle the inclusion state
-              if (_selectedviews.indexOf(p) == -1)
+              if (_selectedviews.indexOf(ap) == -1)
               {
-                _selectedviews.add(p);
-                a.setSelected(true);
+                _selectedviews.add(ap);
+                checkBox.setSelected(true);
               }
               else
               {
-                _selectedviews.remove(p);
-                a.setSelected(false);
+                _selectedviews.remove(ap);
+                checkBox.setSelected(false);
               }
               enabled = true;
               _handler.itemStateChanged(e);
@@ -250,13 +252,13 @@ public class ViewSelectionMenu extends JMenu
             {
               // Deselect everything and select this item only
               _selectedviews.clear();
-              _selectedviews.add(p);
+              _selectedviews.add(ap);
               enabled = false;
               for (Component c : getMenuComponents())
               {
                 if (c instanceof JCheckBoxMenuItem)
                 {
-                  ((JCheckBoxMenuItem) c).setSelected(a == c);
+                  ((JCheckBoxMenuItem) c).setSelected(checkBox == c);
                 }
               }
               enabled = true;
@@ -266,54 +268,31 @@ public class ViewSelectionMenu extends JMenu
           }
         }
       });
-      a.addMouseListener(new MouseListener()
+      checkBox.addMouseListener(new MouseAdapter()
       {
-
-        @Override
-        public void mouseReleased(MouseEvent e)
-        {
-          // TODO Auto-generated method stub
-
-        }
-
-        @Override
-        public void mousePressed(MouseEvent e)
-        {
-          // TODO Auto-generated method stub
-
-        }
-
         @Override
         public void mouseExited(MouseEvent e)
         {
           try
           {
-            p.setSelected(false);
+            ap.setSelected(false);
           } catch (Exception ex)
           {
           }
-          ;
         }
 
         @Override
         public void mouseEntered(MouseEvent e)
         {
-
           try
           {
-            p.setSelected(true);
+            ap.setSelected(true);
           } catch (Exception ex)
           {
           }
-          ;
-        }
-
-        @Override
-        public void mouseClicked(MouseEvent e)
-        {
         }
       });
-      add(a);
+      add(checkBox);
     }
   }