Swing JTextField in JWindow, JWindow bekommt focus zurück?

Harry Kane

Top Contributor
Hallo,
für eine Swing GUI, die sehr kompakt sein soll, brauche ich kleine JTextField, die wen sie den Focus bekommen, eine größeres, ev. mehrzeiliges Eingabefeld anzeigen. Dazu habe ich mir eine eigene Klasse TextPopupWin gebastelt, die so funktionieren soll:
das kleine JTextField bekommt den Focus
am Ort des kleinen JTextFields wird ein JWindow mit einer grösseren JTextArea eingeblendet
auf der JTextArea wird ein FocusListener registriert, der, wenn die JTextArea den Focus verliert (z. B. weil der user ausserhalb von JTextArea/JWindow wohin klickt)
auf dem JWindow wird ein WindowFocusListener registiert, der wenn das JWindow den Focus verliert, dieses schliesst.
Komischerweise muss ich zweimal ausserhalb von JTextArea/JWindow klicken, damit das JWindow geschlossen wird. Ein Wechsal von einem textPopupWin zum nächsten klappt komischerweise.
Hier der Code. Vielen Dank fürs Anschauen und für Tipps.
Java:
import java.awt.*;
import java.awt.event.*;
import java.util.Locale;
import javax.swing.*;

public class TextPopupDemo{

    public static void main(String[] args){
          JFrame frame = new JFrame("Integer Document Demo");
          Box box = new Box(BoxLayout.Y_AXIS);
          Locale.setDefault(Locale.US);
          box.add(new JTextField("Click here to change focus"));
          box.add(new JLabel("With JWindow"));
          final TextPopupWin tp = new TextPopupWin("3.14");
          box.add(tp);
          box.add(new JLabel("Another"));
          final TextPopupWin tp1 = new TextPopupWin("3.14");
          box.add(tp1);
          frame.getContentPane().add(box);
          frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          frame.pack();
          frame.setVisible(true);
    }
}

class TextPopupWin extends JTextField implements FocusListener{
     JWindow popup;
     private boolean popupVisible;

     private JTextArea gross = new JTextArea("Erstes", 3, 12);

     TextPopupWin(String text){
          super(text);
          setEditable(false);
          addFocusListener(this);
          Box box = new Box(BoxLayout.Y_AXIS);
          box.add(new JTextField("Zweites", 20));
          gross.addFocusListener(this);
     }
     public void focusGained(FocusEvent fe){
          if(fe.getComponent() == gross){
               System.out.println("textfield gained focus  at " + System.currentTimeMillis());
               return;
          }
          Window ancestor = SwingUtilities.getWindowAncestor(this);
          popup = new JWindow(ancestor);
          popup.add(gross);
          popup.pack();
          popup.setLocation(this.getLocationOnScreen());
          popup.setVisible(true);
          popupVisible = true;
          popup.addWindowFocusListener(new WindowFocusListener(){
               public void windowGainedFocus(WindowEvent we){
                    System.out.println("windowGainedFocus at " + System.currentTimeMillis());
                    System.out.println("PopupVisible " + popupVisible);
               }
               public void windowLostFocus(WindowEvent we){
                    System.out.println("windowLostFocus at " + System.currentTimeMillis());
                    System.out.println("PopupVisible " + popupVisible);
                    if(popupVisible){
                         popupVisible = false;
                         popup.setVisible(false);
                    }
               }
          });
     }

     public void focusLost(FocusEvent fe){
          if(fe.getComponent() == gross && popupVisible){
               System.out.println("textfield lost focus at " + System.currentTimeMillis());
               popupVisible = false;
               popup.setVisible(false);
          }
     }
}
 
Sowohl das "kleine" JTextField braucht einen FocusListener (um bei Erhalt des Focus die grosse JTextArea einzublende) als auch die "grosse" JTextArea (damit bei Verlust des Focus das JWindow geschlossen werden kann).
 
Du solltest in deiner focusGained Methode nicht jedes Mal ein neues Window machen. Das ist schlechtes uneffizientes Design. Mach dir dein popupwindow im Konstruktor gleich fertig und ruf im focusGained/Lost nur setVisible auf. Vielleicht behebt das schon dein Problem weil du dadurch weniger Windows aufmachst und den EDT dadurch nicht so zuspammst.
 
Du solltest in deiner focusGained Methode nicht jedes Mal ein neues Window machen. Das ist schlechtes uneffizientes Design. Mach dir dein popupwindow im Konstruktor gleich fertig und ruf im focusGained/Lost nur setVisible auf. Vielleicht behebt das schon dein Problem weil du dadurch weniger Windows aufmachst und den EDT dadurch nicht so zuspammst.
Wurde schon getestet und verworfen. Aus einem anderen Beitrag in den Tiefen des webs habe ich entnommen, dass ein JWindow nur focussierbar ist, wenn es einen sichtbaren owner hat. Der owner des JWindow-popups ist natürlich der Window-Ancestor meines Textfeldes. Und dieser ancestor ist im Konstruktor des Textfeldes noch null, weil zum Zeitpunkt der Konstruktion das Textfeld noch zu keinem Container gehört. Ein nachträgliches Setzen des owner geht offenbar auch nicht, zumindest habe ich in JWindow und Window kein setOwner(Window owner) finden könne. Bei Window gibt es zumindest getOwner().
Ich vermute aber dass ich den Fehler erkannt habe: wenn ich auf Panel mit meinem Textfeld klicke, verliert das Textfeld im JWindow-popup den focus und das popup schliesst sich. Beim ersten Klick auf den Panel bekommt aber offenbar die Komponente, die zuletzt fokussiert war, den Focus zurück. Und die zuletzt fokussierte Komponente war mein Textfeld, das aufgrund des erneuten focusGained ein neues popup einblendet.
ich habs mit einem "popupHadFocus" flag gelöst, das beim Einblenden des popups auf true gesetzt wird. Bekommt mein Textfeld den Focus, wird geprüft, ob das flag auf true steht. Wenn ja, wird es auf false gesetzt und sonst nix gemacht, wenn es auf false steht, wird ein neues popup angezeigt.
Doof an der JWindow-Lösung ist, dass das popup sichtbar bleibt, wenn ich nicht auf eine Komponente auf dem Panel im Hauptfenster klicke, sondern auf den Rahmen des Hauptfenster.
Ich habe dann mit einem JPopupWindow rumprobiert. Dabei verhielt sich zunächst auch nicht wie erwartet, da offenbar mein TextFeld bei Klick auf den panel aus irgendwelchen Gründen zweimal unmittelbar hintereinander den focus bekam. Das liess sich aber durch eine Variable lösen, die einen zu schnellen 2. Focusgewinn ins Leere laufen lässt.

Hier der Code für beide Ansätze (JWindow und JPopupWindows). Heute gings, mal schauen wie es morgen aussieht:
Java:
package swingawt;

import java.awt.*;
import java.awt.event.*;
import java.util.Locale;
import javax.swing.*;
import javax.swing.event.*;

public class TextPopupDemo {

    public static void main(String[] args) {
        JFrame frame = new JFrame("Integer Document Demo");
        Box box = new Box(BoxLayout.Y_AXIS);
        Locale.setDefault(Locale.US);
        box.add(new JTextField("Click here to change focus"));
        box.add(new JLabel("With JWindow"));
        final TextPopupWin tp = new TextPopupWin("3.14");
        box.add(tp);
        box.add(new JLabel("Another"));
        final TextPopupWin tp1 = new TextPopupWin("3.14");
        box.add(tp1);
        box.add(new JLabel("Some"));
        box.add(new JLabel("Useless"));
        box.add(new JLabel("Labels"));
        box.add(new JLabel("TextPopupJPopup"));
        final TextPopupJPopup tpj1 = new TextPopupJPopup("3.14");
        box.add(tpj1);
        frame.getContentPane().add(box);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
    }
}

class TextPopupWin extends JTextField implements FocusListener {

    JWindow popup;
    private boolean popupHadFocus;
    private JTextArea gross = new JTextArea("Erstes", 3, 12);

    TextPopupWin(String text) {
        super(text);
        setEditable(false);
        addFocusListener(this);
        Box box = new Box(BoxLayout.Y_AXIS);
        box.add(new JTextField("Zweites", 20));
        gross.addFocusListener(this);
    }

    public void focusGained(FocusEvent fe) {
        if (fe.getComponent() == gross) {
            System.out.println("textfield gained focus  at " + System.currentTimeMillis());
            return;
        }
        if (fe.getComponent() == this) {
            if (popupHadFocus) {
                popupHadFocus = false;
                return;
            }
            System.out.println("textpopup got focus at  " + System.currentTimeMillis());
            Window ancestor = SwingUtilities.getWindowAncestor(this);
            popup = new JWindow(ancestor);
            popup.add(gross);
            popup.pack();
            popup.setLocation(this.getLocationOnScreen());
            popup.setVisible(true);
            gross.requestFocus();
            popup.addWindowFocusListener(new WindowFocusListener() {

                public void windowGainedFocus(WindowEvent we) {
                    System.out.println("windowGainedFocus at " + System.currentTimeMillis());
                    popupHadFocus = true;
                }

                public void windowLostFocus(WindowEvent we) {
                    System.out.println("windowLostFocus at " + System.currentTimeMillis());
                    popup.dispose();
                }
            });
        }
    }

    public void focusLost(FocusEvent fe) {
        if (fe.getComponent() == gross) {// && popupVisible){
            System.out.println("textfield lost focus at " + System.currentTimeMillis());
            popup.dispose();
        }
    }
}

class TextPopupJPopup extends JTextField implements FocusListener {

    JPopupMenu popup;

    private boolean popupHadFocus;

    private JTextArea gross = new JTextArea("Erstes", 3, 12);

    private long lastFocused;

    TextPopupJPopup(String text) {
        super(text);
        setEditable(false);
        addFocusListener(this);
        Box box = new Box(BoxLayout.Y_AXIS);
        box.add(new JTextField("Zweites", 20));
        gross.addFocusListener(this);
        popup = new JPopupMenu();
        popup.add(gross);
        popup.pack();
        popup.addPopupMenuListener(new PopupMenuListener() {

            public void popupMenuCanceled(PopupMenuEvent pe) {
                System.out.println("popupmeuCanceled at " + System.currentTimeMillis());
                System.out.println("PopupVisible " + popupHadFocus);
                //popup.setVisible(false);
                //popupHadFocus = false;
            }

            public void popupMenuWillBecomeVisible(PopupMenuEvent pe) {
                System.out.println("popupMenuWillBecomeVisible at " + System.currentTimeMillis());
                System.out.println("PopupVisible " + popupHadFocus);
                //popupHadFocus = true;
            }

            public void popupMenuWillBecomeInvisible(PopupMenuEvent pe) {
                System.out.println("popupMenuWillBecomeInvisible at " + System.currentTimeMillis());
                System.out.println("PopupVisible " + popupHadFocus);
            }
        });
    }

    public void focusGained(FocusEvent fe) {
        long now = System.currentTimeMillis();
        if((now - lastFocused) < 50) return;
        if (fe.getComponent() == gross) {
            System.out.println("textfield gained focus  at " + now);
            lastFocused = now;
            return;
        }
        if (popupHadFocus) {
            System.out.println("text popup got focus at " + System.currentTimeMillis());
            popupHadFocus = false;
            lastFocused = now;
            return;
        }
        System.out.println("text popup got focus at " + System.currentTimeMillis() + ", creating new popup");
        popup.show(this, 0, 0);
        popup.setVisible(true);
        gross.requestFocus(true);
        popupHadFocus = true;
    }

    public void focusLost(FocusEvent fe) {
        if (fe.getComponent() == gross) {
            System.out.println("textfield lost focus at " + System.currentTimeMillis());
            //popupHadFocus = false;
            popup.setVisible(false);
        }
    }
}
 
Der owner des JWindow-popups ist natürlich der Window-Ancestor meines Textfeldes. Und dieser ancestor ist im Konstruktor des Textfeldes noch null
WTFrack? Du kannst mit getParent beziehungsweise getTopLevelAncestor jederzeit den "Chefkoch" deines Components herausfinden.
 
WTFrack? Du kannst mit getParent beziehungsweise getTopLevelAncestor jederzeit den "Chefkoch" deines Components herausfinden.
Schon klar. Leider ist im Konstruktor meiner Component der Parent noch null, weil die Component in der Phase ihrer Entstehung noch keinem Container bzw. Parent zugeordnet wurde. Und ohne parent oder top level window meiner Component kann ich den owner des JWindow-popups nicht setzen. Und wenn das JWindow-popup keinen owner hat, ist es nicht fokussierbar, und ich kann in das Textfeld im JWindow nichts eingeben. Alles in allem ist der Ansatz "das poup im Konstruktor gleich fertig zu machen" nicht zielführend.
Was mir gerade einfällt: das popup in der focusGained methode lazy zu initialisieren.
 
Ich weiß echt nicht warum du es dir so schwer machst. Wenn du im Konstruktor den Owner deines Textfeldes benötigst, dann lass dir den Owner im Konstruktor übergeben.
Wenn du das nicht willst, könntest du immer noch einen HierarchyChanged Listener an dein Textfeld hängen, welcher dann feuert, wenn dein Textfeldn einen neuen Owner bekommt und mit diesem dann ein neues PopupWindow erzeugt.
 
Wenn du im Konstruktor den Owner deines Textfeldes benötigst, dann lass dir den Owner im Konstruktor übergeben.
Gut, ich könnte natürlich meinem TextPopup einen Konstruktor spendieren, der einen Container oder Window als Parameter akzeptiert und damit das JWindow erzeugen. Leider werden die TextPopups (wie ein Haufen weiterer JComponents auch) per Factory erzeugt, die von Containern nix weiss (die Factory kann nur anhand eine Objekttyps eine JComponent zur Anzeige/Änderung des Objektinhalts ermitteln). Und das zu ändern, nur um diesen Spezialfall zu behandeln, ist zu aufwändig.

Wenn du das nicht willst, könntest du immer noch einen HierarchyChanged Listener an dein Textfeld hängen, welcher dann feuert, wenn dein Textfeldn einen neuen Owner bekommt und mit diesem dann ein neues PopupWindow erzeugt.
Ich glaube ich bleibe bei der lazy initialization🙂. Die kann zwar mit geänderten containment hierarchies nicht umgehen, mit einer Änderung der containment hierarchy ist aber eh nicht zu rechnen. Dafür ist die lazy intialization fast unschlagbar kurz. Also kein Grund für Kommentare wie
Ich weiß echt nicht warum du es dir so schwer machst.

Ausserdem fällt mir gerade auf, dass die ganze Diskussion ohnehin nicht um mein eigentliches problem ging 😀
 
Naja bis auf das klappt der Code eigentlich bei mir. Oder möchtest du, dass das Window schließt wenn man irgendwo auf den Owner Frame hinklickt? Das liegt wohl daran, dass der Focus nicht angenommen wird und sofort wieder zurück zum Textfeld geht, der wieder ein neues Window aufmacht. Evtl. musst du die FocusTraversalPolicy abändern.
 

Neue Themen


Zurück
Oben