Factored out TableCell setEnable() code

cmaglie · cmaglie · commit 015f658d1ec8 · 2015-12-29T16:01:33.000+01:00
diff --git a/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCell.java b/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCell.java
@@ -239,5 +239,9 @@ private static JTextPane makeNewDescription(JPanel panel) {
     return description;
   }
 
-
+  public void setButtonsVisible(boolean enabled) {
+    installButton.setEnabled(enabled);
+    buttonsPanel.setVisible(enabled);
+    inactiveButtonsPanel.setVisible(!enabled);
+  }
 }
diff --git a/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellEditor.java b/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellEditor.java
@@ -141,9 +141,7 @@ public Component getTableCellEditorComponent(JTable table, Object value,
 
   @Override
   public void setEnabled(boolean enabled) {
-    editorCell.installButton.setEnabled(enabled);
-    editorCell.buttonsPanel.setVisible(enabled);
-    editorCell.inactiveButtonsPanel.setVisible(!enabled);
+    editorCell.setButtonsVisible(enabled);
   }
 
   public void setStatus(String status) {
diff --git a/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellRenderer.java b/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellRenderer.java
@@ -44,10 +44,7 @@ public Component getTableCellRendererComponent(JTable table, Object value,
                                                  int column) {
 
     ContributedLibraryTableCell cell = new ContributedLibraryTableCell();
-    cell.installButton.setEnabled(false);
-    cell.buttonsPanel.setVisible(false);
-    cell.inactiveButtonsPanel.setVisible(true);
-
+    cell.setButtonsVisible(false);
     cell.update(table, value, isSelected, row, false);
 
     if (row % 2 == 0) {
diff --git a/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCell.java b/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCell.java
@@ -278,4 +278,11 @@ private static JTextPane makeNewDescription(JPanel panel) {
     return description;
   }
 
+  public void setButtonsVisible(boolean enabled) {
+    installButton.setEnabled(enabled);
+    removeButton.setEnabled(enabled);
+    buttonsPanel.setVisible(enabled);
+    inactiveButtonsPanel.setVisible(!enabled);
+  }
+
 }
diff --git a/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellEditor.java b/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellEditor.java
@@ -138,10 +138,7 @@ public Component getTableCellEditorComponent(JTable table, Object value,
 
   @Override
   public void setEnabled(boolean enabled) {
-    editorCell.installButton.setEnabled(enabled);
-    editorCell.removeButton.setEnabled(enabled);
-    editorCell.buttonsPanel.setVisible(enabled);
-    editorCell.inactiveButtonsPanel.setVisible(!enabled);
+    editorCell.setButtonsVisible(enabled);
   }
 
   public void setStatus(String status) {
diff --git a/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellRenderer.java b/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellRenderer.java
@@ -43,10 +43,7 @@ public Component getTableCellRendererComponent(JTable table, Object value,
                                                  boolean hasFocus, int row,
                                                  int column) {
     ContributedPlatformTableCell cell = new ContributedPlatformTableCell();
-    cell.installButton.setEnabled(false);
-    cell.removeButton.setEnabled(false);
-    cell.buttonsPanel.setVisible(false);
-    cell.inactiveButtonsPanel.setVisible(true);
+    cell.setButtonsVisible(false);
     
     cell.update(table, value, isSelected, row, false);
     if (row % 2 == 0) {

Original file line number	Diff line number	Diff line change
`@@ -239,5 +239,9 @@ private static JTextPane makeNewDescription(JPanel panel) {`
`239`	`239`	`return description;`
`240`	`240`	`}`
`241`	`241`
`242`		`-`
	`242`	`+ public void setButtonsVisible(boolean enabled) {`
	`243`	`+ installButton.setEnabled(enabled);`
	`244`	`+ buttonsPanel.setVisible(enabled);`
	`245`	`+ inactiveButtonsPanel.setVisible(!enabled);`
	`246`	`+ }`
`243`	`247`	`}`
Original file line number	Diff line number	Diff line change
`@@ -141,9 +141,7 @@ public Component getTableCellEditorComponent(JTable table, Object value,`
`141`	`141`
`142`	`142`	`@Override`
`143`	`143`	`public void setEnabled(boolean enabled) {`
`144`		`- editorCell.installButton.setEnabled(enabled);`
`145`		`- editorCell.buttonsPanel.setVisible(enabled);`
`146`		`- editorCell.inactiveButtonsPanel.setVisible(!enabled);`
	`144`	`+ editorCell.setButtonsVisible(enabled);`
`147`	`145`	`}`
`148`	`146`
`149`	`147`	`public void setStatus(String status) {`
Original file line number	Diff line number	Diff line change
`@@ -278,4 +278,11 @@ private static JTextPane makeNewDescription(JPanel panel) {`
`278`	`278`	`return description;`
`279`	`279`	`}`
`280`	`280`
	`281`	`+ public void setButtonsVisible(boolean enabled) {`
	`282`	`+ installButton.setEnabled(enabled);`
	`283`	`+ removeButton.setEnabled(enabled);`
	`284`	`+ buttonsPanel.setVisible(enabled);`
	`285`	`+ inactiveButtonsPanel.setVisible(!enabled);`
	`286`	`+ }`
	`287`	`+`
`281`	`288`	`}`