Changeset 5236

Timestamp:

2003-08-21T11:10:38+12:00 (21 years ago)

Author:

jmt12

Message:

Buttons are now enabled/disabled appropriately. Note that you now cannot add a Subcollection Partition that is already defined (ie add is not enabled)

File:

• trunk/gli/src/org/greenstone/gatherer/cdm/SubcollectionIndexManager.java (modified) (12 diffs)

trunk/gli/src/org/greenstone/gatherer/cdm/SubcollectionIndexManager.java

@@ -42 +42 @@
 import org.greenstone.gatherer.util.ExclusiveListSelectionListener;
 import org.greenstone.gatherer.util.StaticStrings;
+import org.greenstone.gatherer.util.Utility;
 import org.w3c.dom.*;
 /** This class maintains a list of indexes partitions for the purpose of defining subcollections.
@@ -235 +236 @@
         clear_default_button = new JButton(get("Clear_Default_Subindex"));
         clear_default_button.setMnemonic(KeyEvent.VK_C);
-        if(default_index == null) {
-        clear_default_button.setEnabled(false);
-        }
+        clear_default_button.setEnabled(default_index != null);
 
         JLabel default_label = new JLabel(get("Default_Subindex"));
+        default_label.setPreferredSize(Utility.LABEL_SIZE);
         JPanel default_pane = new JPanel();
         if(default_index == null) {
@@ -251 +251 @@
         remove_index_button = new JButton(get("Remove_Subindex"));
         remove_index_button.setMnemonic(KeyEvent.VK_R);
+        remove_index_button.setEnabled(false);
         set_default_button = new JButton(get("Set_Default_Subindex"));
         set_default_button.setMnemonic(KeyEvent.VK_S);
+        set_default_button.setEnabled(false);
         JLabel subcollection_label = new JLabel(get("Subcollection"));
         subcollection_list = new JList(CollectionDesignManager.subcollection_manager);
@@ -270 +272 @@
         set_default_button.addActionListener(new SetDefaultSubIndexListener());
 
+        subcollectionindexes_list.addListSelectionListener(new SubcollectionListListener());
+
         subcollectionindex_name_field.getDocument().addDocumentListener(new SubcollectionIndexListener());
         subcollection_list.addListSelectionListener(new SubcollectionIndexListener());
@@ -279 +283 @@
         default_label.setBorder(BorderFactory.createEmptyBorder(0,0,0,5));
 
-        default_pane.setBorder(BorderFactory.createCompoundBorder(BorderFactory.createRaisedBevelBorder(), BorderFactory.createEmptyBorder(2,2,2,2)));
+        default_pane.setBorder(BorderFactory.createCompoundBorder(BorderFactory.createEmptyBorder(2,0,0,0), BorderFactory.createCompoundBorder(BorderFactory.createRaisedBevelBorder(), BorderFactory.createEmptyBorder(2,2,2,2))));
         default_pane.setLayout(new BorderLayout());
         default_pane.add(default_label, BorderLayout.WEST);
@@ -289 +293 @@
         subindexes_pane.add(default_pane, BorderLayout.SOUTH);
         
+        subindex_name_panel.setBorder(BorderFactory.createEmptyBorder(2,0,0,0));
         subindex_name_panel.setLayout(new BorderLayout());
         subindex_name_panel.add(subindex_name_label, BorderLayout.WEST);
@@ -297 +302 @@
         list_pane.add(subcollection_label, BorderLayout.NORTH);
         list_pane.add(new JScrollPane(subcollection_list), BorderLayout.CENTER);
-        
-        button_pane.setLayout(new GridLayout(1,4));
+        list_pane.add(subindex_name_panel, BorderLayout.SOUTH);
+
+        button_pane.setLayout(new GridLayout(2,2));
         button_pane.add(add_index_button);
         button_pane.add(remove_index_button);
+        button_pane.add(set_default_button);
         button_pane.add(clear_default_button);
-        button_pane.add(set_default_button);
-
-        temp_pane.setLayout(new GridLayout(2,1,0,5));
-        temp_pane.add(subindex_name_panel);
-        temp_pane.add(button_pane);
 
         subcollection_panel.setLayout(new BorderLayout());
@@ -312 +314 @@
         subcollection_panel.add(list_pane, BorderLayout.CENTER);
         //subcollection_panel.add(button_pane, BorderLayout.SOUTH);
-        subcollection_panel.add(temp_pane, BorderLayout.SOUTH);
+        subcollection_panel.add(button_pane, BorderLayout.SOUTH);
         
         setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
@@ -356 +358 @@
         public void actionPerformed(ActionEvent event) {
         setDefaultSubcollectionIndex(null);
+        default_value_field.setText("");
         clear_default_button.setEnabled(false);
-        default_value_field.setText("");
+        set_default_button.setEnabled(!subcollectionindexes_list.isSelectionEmpty());
         }
     }
@@ -385 +388 @@
         if(!subcollectionindexes_list.isSelectionEmpty()) {
             setDefaultSubcollectionIndex((SubcollectionIndex)subcollectionindexes_list.getSelectedValue());
+            default_value_field.setText(default_index.toString());
             clear_default_button.setEnabled(true);
-            default_value_field.setText(default_index.toString());
+            set_default_button.setEnabled(false);
         }
         }
@@ -417 +421 @@
         if(!subcollection_list.isSelectionEmpty() && subcollectionindex_name_field.getText().length() > 0) {
             if (getSubcollectionIndex(subcollectionindex_name_field.getText()) == null) {
-            add_index_button.setEnabled(true);
+            SubcollectionIndex subindex = new SubcollectionIndex(subcollection_list.getSelectedValues());
+            add_index_button.setEnabled(!model.contains(subindex));
             } else {
             add_index_button.setEnabled(false);
@@ -427 +432 @@
         }
     }
+
+    private class SubcollectionListListener
+        implements ListSelectionListener {
+
+        public void valueChanged(ListSelectionEvent event) {
+        if(!event.getValueIsAdjusting()) {
+            boolean enable = !subcollectionindexes_list.isSelectionEmpty();
+            remove_index_button.setEnabled(enable);
+            set_default_button.setEnabled(enable);
+        }
+        }
+    }
     }
 }