Changeset 96 in tmcsimulator-scriptbuilder for trunk/src/scriptbuilder/gui

Timestamp:

08/29/2017 09:44:11 AM (9 years ago)

Author:

bmcguffin

Message:

Removed "Delete" option from popup context menu. Incidents can still be deleted from the "Delete Incident" dropdown menu item; this change forces the user to be more deliberate in their decision to delete an incident.

File:

• trunk/src/scriptbuilder/gui/panels/IncidentTimelinePanel.java (modified) (3 diffs)

trunk/src/scriptbuilder/gui/panels/IncidentTimelinePanel.java

@@ -108 +108 @@
             JMenuItem eventsMenuItem = new JMenuItem("Events");
             JMenuItem propsMenuItem = new JMenuItem("Properties");
-            JMenuItem deleteMenuItem = new JMenuItem("Delete");
             eventsMenuItem.setActionCommand("Edit Events");
             propsMenuItem.setActionCommand("Modify Incident Properties");
-            deleteMenuItem.setActionCommand("Delete Incident");
 
             PopupMenuItemListener menuItemListener = new PopupMenuItemListener();
@@ -117 +115 @@
             eventsMenuItem.addActionListener(menuItemListener);
             propsMenuItem.addActionListener(menuItemListener);
-            deleteMenuItem.addActionListener(menuItemListener);
 
             menu.add(eventsMenuItem);
             menu.add(propsMenuItem);
-            menu.add(deleteMenuItem);
             return menu;
         }
@@ -145 +141 @@
                         ((ScriptBuilderFrame) topFrame).incidentDetailsScreen(incident);
                         ((ScriptBuilderFrame) topFrame).update(script, script);
-                    }
-                    if (e.getActionCommand().equals("Delete Incident"))
-                    {
-                        int confirm = JOptionPane.showConfirmDialog(((ScriptBuilderFrame) topFrame),
-                                "Are you sure you want to delete " + incident.toString() + "?");
-                        if (confirm == JOptionPane.YES_OPTION)
-                        {
-                            script.incidents.remove(incident);
-                            script.incidents.add(null);
-                            ((ScriptBuilderFrame) topFrame).update(script, script);
-                        }
                     }
                 }