JPopupMenu не исчезает/перемещается при минимизации/перемещении JFrame в

Question

Я делаю mp3-плеер с несколькими JLists в своем JFrame. Когда я нажимаю правой кнопкой мыши на элемент JList, появляется всплывающее окно с некоторыми параметрами для этой песни. Но когда это всплывающее окно видно, и я минимизирую свой JFrame, это всплывающее окно остается видимым! Кроме того, когда всплывающее окно видимо, и я перетаскиваю свой JFrame в другое место на экране, всплывающее окно остается на его исходной позиции (так что оно не остается в том же положении относительно JFrame) ... Может кто-то, пожалуйста, помогите мне с этим? Я старался как можно больше разбить этот класс :)

Я был бы очень благодарен, если бы кто-то мог мне помочь!

Джо

public class playListPanel extends JPanel implements MouseListener { 
    private DefaultListModel model; 
    private Interface interFace; 
    private JList list; 
    private boolean emptyPlaylist; 
    private ArrayList<Song> currentPlayList; 
    private Song rightClickedSong; 
    private JPopupMenu popup; 
    private Point panelLocation; 

    public playListPanel(Interface interFace) // Interface extends JFrame, 
               // playListPanel is a part of 
               // this JFrame. 
    { 
     this.interFace = interFace; 
     this.panelLocation = new Point(559, 146); 
     setBackground(SystemColor.controlHighlight); 
     setBorder(new TitledBorder(null, "", TitledBorder.LEADING, 
       TitledBorder.TOP, null, null)); 
     setBounds((int) panelLocation.getX(), (int) panelLocation.getY(), 698, 
       368); 
     setLayout(null); 

     currentPlayList = new ArrayList<Song>(); 

     model = new DefaultListModel(); 
     list = new JList(model); 
     list.setVisible(true); 
     list.addMouseListener(this); 

     JScrollPane scrollPane = new JScrollPane(list); 
     scrollPane.setBounds(5, 5, 688, 357); 
     add(scrollPane); 

     emptyPlaylist = true; 
    } 

    private void openMenuPopup(Point point) 
    { 
     removePopup(); 
     popup = new JPopupMenu(); 
     int x = (int) point.getX(); 
     int y = (int) point.getY(); 
     popup.setLocation((int) (x+panelLocation.getX()),(int) (y+panelLocation.getY())); 
     //popup.setLabel("popup voor playlist"); 
     JMenuItem removeSong; 
     popup.add(removeSong = new JMenuItem("Remove Song from Playlist", new ImageIcon("image.jpg"))); 

     ActionListener menuListener = new ActionListener() 
     { 
       public void actionPerformed(ActionEvent event) 
       { 
       if(event.getActionCommand().equals("Remove Song from Playlist")) 
       { 
        System.out.println("Remove Song from Playlist"); 
        interFace.getPlaylistManager().removeOneSong(rightClickedSong); 
        removePopup(); 
       } 

     }; 

     //ADD THE LISTENERS TO THE MENU ITEMS 
     removeSong.addActionListener(menuListener); 
     popup.setVisible(true); 
    } 

    public void removePopup() 
    { 
     if(popup!==null) 
     { 
      popup.setVisible(false); 
      System.out.println("popup removed"); 
     } 
    } 

    private int getRow(Point point) { 
     return list.locationToIndex(point); 
    } 

    public void refreshPlayList(ArrayList<Song> playlist) { 
     this.currentPlayList = playlist; 

     model.clear(); 
     for (Song song : playlist) { 
      model.add(model.getSize(), song.getPlaylistString()); 
     } 
     list.setVisible(true); 
    } 

    public void highlightSong(int index) { 
     list.setSelectedIndex(index); 
    } 

    public int getRowOfList(Point point) { 
     return list.locationToIndex(point); 
    } 

    @Override 
    public void mouseClicked(MouseEvent e) { 
     interFace.getPlaylistManager().doubleClickOnPlaylist(e); 
    } 

    @Override 
    public void mouseEntered(MouseEvent arg0) { 
    } 

    @Override 
    public void mouseExited(MouseEvent arg0) { 
    } 

    @Override 
    public void mousePressed(MouseEvent e) { 
     if (SwingUtilities.isRightMouseButton(e)) { 
      rightClickedSong = currentPlayList.get(getRow(e.getPoint())); 
      openMenuPopup(e.getPoint()); 
      System.out.println("should open popup at " 
        + e.getPoint().toString()); 
     } 

    } 

    @Override 
    public void mouseReleased(MouseEvent arg0) { 
    } 
} 

Answer 1

Есть некоторые основные недостатки в том, как вы обработке нажмите для показа всплывающего окна. В простых сценариях не рекомендуется звонить . Вместо этого вы можете полагаться на свое поведение по умолчанию. Кроме того, лучше использовать e.isPopupTrigger(), чем для проверки SwingUtilities.isRightMouseButton(e), чтобы отобразить всплывающее окно.

Вы можете сделать что-то вроде следующего:

//at classlevel, 
private JPopupMenu popup = new JPopupMenu(); 
//create a Popuplistener 
PopupListener pl = new PopupListener(); 
list.addMouseListener(pl); 

//Implementation of your popuplistener 
    class PopupListener extends MouseAdapter { 
    public void mousePressed(MouseEvent e) { 
     maybeShowPopup(e); 
    } 

    public void mouseReleased(MouseEvent e) { 
     maybeShowPopup(e); 
    } 

    private void maybeShowPopup(MouseEvent e) { 
     if (e.isPopupTrigger()) 
     //e.getSource - and construct your popup as required. 
     //and then. 
     popup.show(((JApplet) e.getComponent()).getContentPane(), e 
      .getX(), e.getY()); 
    } 
    }