From 0e71d7645a86767258d2f15bb59aec966ac47c8f Mon Sep 17 00:00:00 2001 From: George Gastaldi Date: Fri, 11 Dec 2015 11:59:01 -0200 Subject: [PATCH] FORGE-2553: Labels now show tooltip --- .../forge/plugin/idea/ui/component/ChooserComponent.java | 5 ++++- .../plugin/idea/ui/component/ComboComponentBuilder.java | 2 ++ .../forge/plugin/idea/ui/component/LabeledComponent.java | 2 ++ .../jboss/forge/plugin/idea/ui/component/TextComponent.java | 2 +- 4 files changed, 9 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/jboss/forge/plugin/idea/ui/component/ChooserComponent.java b/src/main/java/org/jboss/forge/plugin/idea/ui/component/ChooserComponent.java index 761fc87..fde2160 100644 --- a/src/main/java/org/jboss/forge/plugin/idea/ui/component/ChooserComponent.java +++ b/src/main/java/org/jboss/forge/plugin/idea/ui/component/ChooserComponent.java @@ -55,6 +55,8 @@ public void buildUI(Container container) component = new ComponentWithBrowseButton<>(textField, createBrowseButtonActionListener(textField)); inputField = component.getChildComponent(); + inputField.setToolTipText(input.getDescription()); + component.setToolTipText(input.getDescription()); inputField.getDocument().addDocumentListener(new DocumentListener() { @Override @@ -79,7 +81,8 @@ public void documentChanged(DocumentEvent event) public void updateState() { component.setEnabled(input.isEnabled()); - + component.setToolTipText(input.getDescription()); + inputField.setToolTipText(input.getDescription()); if (!getValue().equals(getInputValue())) { reloadValue(); diff --git a/src/main/java/org/jboss/forge/plugin/idea/ui/component/ComboComponentBuilder.java b/src/main/java/org/jboss/forge/plugin/idea/ui/component/ComboComponentBuilder.java index c386850..2854fa1 100644 --- a/src/main/java/org/jboss/forge/plugin/idea/ui/component/ComboComponentBuilder.java +++ b/src/main/java/org/jboss/forge/plugin/idea/ui/component/ComboComponentBuilder.java @@ -62,6 +62,7 @@ public void itemStateChanged(ItemEvent e) } } }); + combo.setToolTipText(input.getDescription()); addNoteLabel(container, combo).setText(input.getNote()); } @@ -70,6 +71,7 @@ public void itemStateChanged(ItemEvent e) public void updateState() { combo.setEnabled(input.isEnabled()); + combo.setToolTipText(input.getDescription()); if (!getInputValueChoices().equals(getChoices()) || !getInputValue().equals(getValue())) diff --git a/src/main/java/org/jboss/forge/plugin/idea/ui/component/LabeledComponent.java b/src/main/java/org/jboss/forge/plugin/idea/ui/component/LabeledComponent.java index b610e08..78b568a 100644 --- a/src/main/java/org/jboss/forge/plugin/idea/ui/component/LabeledComponent.java +++ b/src/main/java/org/jboss/forge/plugin/idea/ui/component/LabeledComponent.java @@ -38,6 +38,7 @@ public LabeledComponent(InputComponent input, ForgeComponent componen public void buildUI(Container container) { this.label = new JBLabel(InputComponents.getLabelFor(input, true)); + label.setToolTipText(input.getDescription()); container.add(label); component.buildUI(container); } @@ -70,6 +71,7 @@ public void setValueChangeListener(ValueChangeListener valueChangeListener) public void updateState() { label.setEnabled(input.isEnabled()); + label.setToolTipText(input.getDescription()); component.updateState(); } } diff --git a/src/main/java/org/jboss/forge/plugin/idea/ui/component/TextComponent.java b/src/main/java/org/jboss/forge/plugin/idea/ui/component/TextComponent.java index 8bb3241..27d9cf3 100644 --- a/src/main/java/org/jboss/forge/plugin/idea/ui/component/TextComponent.java +++ b/src/main/java/org/jboss/forge/plugin/idea/ui/component/TextComponent.java @@ -74,6 +74,7 @@ public void documentChanged(DocumentEvent event) public void updateState() { component.setEnabled(input.isEnabled()); + component.setToolTipText(input.getDescription()); if (!getValue().equals(getInputValue())) { @@ -83,7 +84,6 @@ public void updateState() { component.setVariants(getCompletions()); } - component.setToolTipText(input.getDescription()); updateNote(component, input.getNote()); }