Add boarders to the teble items so screen readers will read while walking through the table

Joe Wegner · Joe Wegner · commit b9265a42b120 · 2019-09-27T10:29:19.000-04:00
diff --git a/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellJPanel.java b/app/src/cc/arduino/contributions/libraries/ui/ContributedLibraryTableCellJPanel.java
@@ -3,21 +3,12 @@
 import static processing.app.I18n.format;
 import static processing.app.I18n.tr;
 
-import java.awt.Color;
-import java.awt.Component;
-import java.awt.Dimension;
-import java.awt.Insets;
+import java.awt.*;
 import java.util.Optional;
 
-import javax.swing.Box;
-import javax.swing.BoxLayout;
-import javax.swing.JButton;
-import javax.swing.JComboBox;
-import javax.swing.JLabel;
-import javax.swing.JPanel;
-import javax.swing.JTable;
-import javax.swing.JTextPane;
+import javax.swing.*;
 import javax.swing.border.EmptyBorder;
+import javax.swing.border.TitledBorder;
 import javax.swing.event.HyperlinkEvent;
 import javax.swing.text.Document;
 import javax.swing.text.html.HTMLDocument;
@@ -128,6 +119,9 @@ public ContributedLibraryTableCellJPanel(JTable parentTable, Object value,
       return;
 
     ContributedLibrary selected = releases.getSelected();
+    TitledBorder titledBorder = BorderFactory.createTitledBorder(selected.getName());
+    titledBorder.setTitleFont(getFont().deriveFont(Font.BOLD));
+    setBorder(titledBorder);
     Optional<ContributedLibrary> mayInstalled = releases.getInstalled();
 
     boolean installable, upgradable;
diff --git a/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellJPanel.java b/app/src/cc/arduino/contributions/packages/ui/ContributedPlatformTableCellJPanel.java
@@ -189,7 +189,7 @@ void update(JTable parentTable, Object value, boolean isSelected,
     ContributedPlatform selected = releases.getSelected();
     TitledBorder titledBorder = BorderFactory.createTitledBorder(selected.getName());
     titledBorder.setTitleFont(getFont().deriveFont(Font.BOLD));
-    this.setBorder(titledBorder);
+    setBorder(titledBorder);
     ContributedPlatform installed = releases.getInstalled();
 
     boolean removable, installable, upgradable;
@@ -215,7 +215,7 @@ void update(JTable parentTable, Object value, boolean isSelected,
     removeButtonPlaceholder.setVisible(!removable);
 
     String desc = "<html><body>";
-//    desc += "<b>" + selected.getName() + "</b>";
+    desc += "<b>" + selected.getName() + "</b>";
     if (installed != null && installed.isBuiltIn()) {
       desc += " Built-In ";
     }
