Merge lp:~tapaal-contributor/tapaal/select-all-1763193 into lp:tapaal

Proposed by Jacob Hjort Bundgaard
Status: Merged
Approved by: Jiri Srba
Approved revision: 954
Merged at revision: 956
Proposed branch: lp:~tapaal-contributor/tapaal/select-all-1763193
Merge into: lp:tapaal
Diff against target: 119 lines (+44/-1)
3 files modified
src/pipe/gui/GuiFrame.java (+26/-0)
src/pipe/gui/SelectionManager.java (+9/-1)
src/pipe/gui/graphicElements/PetriNetObject.java (+9/-0)
To merge this branch: bzr merge lp:~tapaal-contributor/tapaal/select-all-1763193
Reviewer Review Type Date Requested Status
Jiri Srba Approve
Review via email: mp+343056@code.launchpad.net

Commit message

Add menu action for selecting all components.

To post a comment you must log in.
954. By Jiri Srba <email address hidden>

merged in trunk

Revision history for this message
Jiri Srba (srba) wrote :

Really great that ctrl-A works now. Thanks for the fix.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/pipe/gui/GuiFrame.java'
2--- src/pipe/gui/GuiFrame.java 2018-04-16 11:54:09 +0000
3+++ src/pipe/gui/GuiFrame.java 2018-04-16 12:16:41 +0000
4@@ -132,6 +132,7 @@
5 private ZoomAction zoomOutAction, zoomInAction;
6 private SpacingAction incSpacingAction, decSpacingAction;
7 private DeleteAction deleteAction;
8+ private SelectAllAction selectAllAction;
9 private TypeAction annotationAction, arcAction, inhibarcAction,
10 placeAction, transAction, timedtransAction, tokenAction,
11 selectAction, deleteTokenAction, timedPlaceAction;
12@@ -521,6 +522,15 @@
13 editMenu.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(
14 KeyStroke.getKeyStroke("BACK_SPACE"), "Delete");
15 editMenu.getActionMap().put("Delete", deleteAction);
16+
17+ editMenu.addSeparator();
18+
19+ addMenuItem(editMenu, selectAllAction = new SelectAllAction("Select all",
20+ "Select all components", "ctrl A"));
21+
22+ editMenu.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(
23+ KeyStroke.getKeyStroke('A', shortcutkey), "SelectAll");
24+ editMenu.getActionMap().put("SelectAll", selectAllAction);
25
26 /* Draw menu */
27 JMenu drawMenu = new JMenu("Draw");
28@@ -1032,6 +1042,7 @@
29 transAction.setEnabled(true);
30 tokenAction.setEnabled(true);
31 deleteAction.setEnabled(true);
32+ selectAllAction.setEnabled(true);
33 selectAction.setEnabled(true);
34 deleteTokenAction.setEnabled(true);
35
36@@ -1074,6 +1085,7 @@
37 transAction.setEnabled(false);
38 tokenAction.setEnabled(false);
39 deleteAction.setEnabled(false);
40+ selectAllAction.setEnabled(false);
41 selectAction.setEnabled(false);
42 deleteTokenAction.setEnabled(false);
43
44@@ -1127,6 +1139,7 @@
45 transAction.setEnabled(false);
46 tokenAction.setEnabled(false);
47 deleteAction.setEnabled(false);
48+ selectAllAction.setEnabled(false);
49 selectAction.setEnabled(false);
50 deleteTokenAction.setEnabled(false);
51
52@@ -2197,6 +2210,19 @@
53
54 }
55
56+ class SelectAllAction extends GuiAction {
57+
58+ private static final long serialVersionUID = -9223372036854775808L;
59+
60+ SelectAllAction(String name, String tooltip, String keystroke) {
61+ super(name, tooltip, keystroke);
62+ }
63+
64+ public void actionPerformed(ActionEvent e) {
65+ CreateGui.getView().getSelectionObject().selectAll();
66+ }
67+ }
68+
69 class TypeAction extends GuiAction {
70
71 /**
72
73=== modified file 'src/pipe/gui/SelectionManager.java'
74--- src/pipe/gui/SelectionManager.java 2016-02-05 09:19:04 +0000
75+++ src/pipe/gui/SelectionManager.java 2018-04-16 12:16:41 +0000
76@@ -179,7 +179,7 @@
77 *
78 * @see java.awt.event.MouseListener#mousePressed(java.awt.event.MouseEvent)
79 */
80- public void mousePressed(MouseEvent e) {
81+ public void mousePressed(MouseEvent e) {
82 CreateGui.getCurrentTab().removeConstantHighlights();
83 if (e.getButton() == MouseEvent.BUTTON1 && !(e.isControlDown())) {
84 isSelecting = true;
85@@ -264,6 +264,14 @@
86 public void mouseMoved(MouseEvent e) {
87 }
88
89+ public void selectAll() {
90+ ArrayList<PetriNetObject> pnObjects = drawingSurface.getPNObjects();
91+ for (PetriNetObject pnObject : pnObjects) {
92+ pnObject.select(false);
93+ }
94+ drawingSurface.repaint();
95+ }
96+
97 public int getSelectionCount() {
98 Component[] netObj = drawingSurface.getComponents();
99 int selectionCount = 0;
100
101=== modified file 'src/pipe/gui/graphicElements/PetriNetObject.java'
102--- src/pipe/gui/graphicElements/PetriNetObject.java 2012-03-02 13:39:05 +0000
103+++ src/pipe/gui/graphicElements/PetriNetObject.java 2018-04-16 12:16:41 +0000
104@@ -100,6 +100,15 @@
105 }
106 }
107
108+ public void select(boolean shouldRepaint) {
109+ if (selectable && !selected) {
110+ selected = true;
111+ if (shouldRepaint) {
112+ repaint();
113+ }
114+ }
115+ }
116+
117 public void deselect() {
118 if (selected) {
119 selected = false;

Subscribers

People subscribed via source and target branches