external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-20 04:38:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

Prevent random crash that sometimes occurs when moving mouse over deck tables

Browse Source
This commit is contained in:
drdev
2014-01-11 04:15:04 +00:00
parent be47a4bad3
commit 496b12913d
1 changed files with 22 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
forge-gui/src/main/java/forge/gui/toolbox/itemmanager/views/ItemListView.java
View File

@@ -328,14 +328,32 @@ public final class ItemListView<T extends InventoryItem> extends ItemView<T> {
if (renderer != null) { if (renderer != null) {
renderer.processMouseEvent(e, ItemListView.this, val, row, col); //give renderer a chance to process the mouse event renderer.processMouseEvent(e, ItemListView.this, val, row, col); //give renderer a chance to process the mouse event
} }
try {
super.processMouseEvent(e); super.processMouseEvent(e);
} }
catch (Exception ex) { //trap error thrown by weird tooltip issue
ex.printStackTrace();
}
}
private String getCellTooltip(TableCellRenderer renderer, int row, int col, Object val) { private String getCellTooltip(TableCellRenderer renderer, int row, int col, Object val) {
Component cell = renderer.getTableCellRendererComponent(this, val, false, false, row, col); Component cell = renderer.getTableCellRendererComponent(this, val, false, false, row, col);
// use a pre-set tooltip if it exists
if (cell instanceof JComponent) {
JComponent jcell = (JComponent)cell;
String tip = jcell.getToolTipText();
if (tip != null && !tip.isEmpty()) {
return tip;
}
}
// if we're conditionally showing the tooltip, check to see // if we're conditionally showing the tooltip, check to see
// if we shouldn't show it // if we shouldn't show it
if (val == null) { return null; }
String text = val.toString();
if (text.isEmpty()) { return null; }
if (!(renderer instanceof ItemCellRenderer) || !((ItemCellRenderer)renderer).alwaysShowTooltip()) { if (!(renderer instanceof ItemCellRenderer) || !((ItemCellRenderer)renderer).alwaysShowTooltip()) {
// if there's enough room (or there's no value), no tooltip // if there's enough room (or there's no value), no tooltip
// we use '>' here instead of '>=' since that seems to be the // we use '>' here instead of '>=' since that seems to be the
@@ -343,22 +361,13 @@ public final class ItemListView<T extends InventoryItem> extends ItemView<T> {
// JTable renderer // JTable renderer
int requiredWidth = cell.getPreferredSize().width; int requiredWidth = cell.getPreferredSize().width;
TableColumn tableColumn = this.getColumnModel().getColumn(col); TableColumn tableColumn = this.getColumnModel().getColumn(col);
if (null == val || tableColumn.getWidth() > requiredWidth) { if (tableColumn.getWidth() > requiredWidth) {
return null; return null;
} }
} }
// use a pre-set tooltip if it exists // if above checks passed, show the full text in the tooltip
if (cell instanceof JComponent) { return text;
JComponent jcell = (JComponent)cell;
String tip = jcell.getToolTipText();
if (tip != null) {
return tip;
}
}
// otherwise, show the full text in the tooltip
return String.valueOf(val);
} }
@Override @Override