Skip to content

Commit 22224df

Browse files
committed
Merge branch 'master' of https://github.com/UCSDOalads/JavaSketchPad into addCDEFbykaic
2 parents d3c882f + 2b612f7 commit 22224df

2 files changed

Lines changed: 20 additions & 17 deletions

File tree

src/ui/helper/historyui/HistoryUI.java

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,33 +3,29 @@
33
import java.awt.BorderLayout;
44
import java.awt.event.ActionEvent;
55
import java.awt.event.ActionListener;
6-
import java.util.ArrayList;
6+
77

88
import javax.swing.JFrame;
99
import javax.swing.JPanel;
1010
import javax.swing.JScrollPane;
1111
import javax.swing.JTable;
12-
import javax.swing.JTextField;
1312
import javax.swing.ListSelectionModel;
14-
import javax.swing.SwingUtilities;
15-
import javax.swing.event.DocumentEvent;
16-
import javax.swing.event.DocumentListener;
1713
import javax.swing.table.DefaultTableModel;
18-
import javax.swing.table.TableModel;
1914

20-
import classpathutil.ClassSearch;
2115

2216
import javax.swing.DefaultListSelectionModel;
2317
import javax.swing.JButton;
2418

2519
public class HistoryUI extends JFrame
2620
{
21+
/**
22+
*
23+
*/
24+
private static final long serialVersionUID = 8141494344180865577L;
2725
private JTable resultsTable;
28-
private ClassSearch searchUtil;
2926
private DefaultTableModel defaultTableModel;
3027
private HistoryUIInterface delegate;
3128

32-
private Thread runningThread;
3329
JButton btnCancel, btnConfirm;
3430
JPanel panel;
3531
JScrollPane scrollPane;
@@ -42,6 +38,11 @@ public HistoryUI(String[] titles) {
4238

4339
// set the defaultTableModel to non editable by user clicking around
4440
defaultTableModel = new DefaultTableModel(0, 1) {
41+
/**
42+
*
43+
*/
44+
private static final long serialVersionUID = 1L;
45+
4546
@Override
4647
public boolean isCellEditable(int row, int column) {
4748
return false;
@@ -57,9 +58,6 @@ public boolean isCellEditable(int row, int column) {
5758
getContentPane().add(panel, BorderLayout.CENTER);
5859
panel.setLayout(new BorderLayout(0, 0));
5960

60-
// search text field on top
61-
searchUtil = ClassSearch.sharedInstance();
62-
6361
// show result
6462
resultsTable = new JTable();
6563
resultsTable.setModel(defaultTableModel);
@@ -70,13 +68,13 @@ public boolean isCellEditable(int row, int column) {
7068
panel.add(scrollPane, BorderLayout.CENTER);
7169

7270
// add small JPanel for Cancel and Confirm buttons
73-
JPanel panel_1 = new JPanel();
74-
getContentPane().add(panel_1, BorderLayout.SOUTH);
71+
JPanel buttom_panel = new JPanel();
72+
getContentPane().add(buttom_panel, BorderLayout.SOUTH);
7573

7674
//create buttons from array list
7775
for (String title : titles){
7876
JButton button = new JButton(title);
79-
panel_1.add(button);
77+
buttom_panel.add(button);
8078
button.addActionListener(new ActionListener() {
8179

8280
@Override
@@ -100,11 +98,16 @@ public void setDelegate(HistoryUIInterface delegate) {
10098
}
10199

102100
/*
103-
* Forcely remove single selection in JTable
101+
* Forcedly remove single selection in JTable
104102
*/
105103
private static class ForcedListSelectionModel
106104
extends
107105
DefaultListSelectionModel {
106+
/**
107+
*
108+
*/
109+
private static final long serialVersionUID = 1L;
110+
108111
public ForcedListSelectionModel() {
109112
this.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
110113
};

src/ui/helper/historyui/HistoryUITest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package ui.helper.historyui;
22

3-
import static org.junit.Assert.*;
3+
44

55
import java.awt.Dimension;
66

0 commit comments

Comments
 (0)