hide confirmations in tooltip when showing inputs and outputs on the transactions table
What changed, and why it matters
This commit is a minor user-interface tweak. It stops showing the number of blockchain confirmations inside a tooltip when the user is viewing the detailed inputs and outputs of a transaction. There is no security issue here.
No security action needed. Treat as a normal UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
CoinCell.java renders amounts in a tree-table. Its tooltip can append a confirmations description. The patch adds a hideConfirmations() method that disables that description and unbinds the confirmations property, then calls it for HashIndexEntry rows (transaction inputs/outputs). This is purely presentational.
Changed components
src/main/java/com/sparrowwallet/sparrow/control/CoinCell.javaInspect captured patch +10 / −0
diff --git a/src/main/java/com/sparrowwallet/sparrow/control/CoinCell.java b/src/main/java/com/sparrowwallet/sparrow/control/CoinCell.java
index 8c7fe5a..12375a5 100644
--- a/src/main/java/com/sparrowwallet/sparrow/control/CoinCell.java
+++ b/src/main/java/com/sparrowwallet/sparrow/control/CoinCell.java
@@ -87,6 +87,8 @@ class CoinCell extends TreeTableCell<Entry, Number> implements ConfirmationsList
} else if(entry instanceof UtxoEntry) {
setGraphic(null);
} else if(entry instanceof HashIndexEntry) {
+ tooltip.hideConfirmations();
+
Region node = new Region();
node.setPrefWidth(10);
setGraphic(node);
@@ -148,6 +150,14 @@ class CoinCell extends TreeTableCell<Entry, Number> implements ConfirmationsList
setTooltipText();
}
+ public void hideConfirmations() {
+ showConfirmations = false;
+ isCoinbase = false;
+ confirmationsProperty.unbind();
+
+ setTooltipText();
+ }
+
private void setTooltipText() {
setText(value + (showConfirmations ? " (" + getConfirmationsDescription() + ")" : ""));
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.