X-Git-Url: http://source.jalview.org/gitweb/?p=jalview.git;a=blobdiff_plain;f=src%2Fjalview%2Fjbgui%2FGDesktop.java;h=e3f171df98d96bd46371d3fdc132fdfec869c6fb;hp=bbde757fc05f5fa5958af55d64de505b0bc33f1e;hb=c9800715ec36a57a6e6afb13a57b927e4061242e;hpb=96026e673c3629afa54a0d03be4b0022c7b42869 diff --git a/src/jalview/jbgui/GDesktop.java b/src/jalview/jbgui/GDesktop.java index bbde757..e3f171d 100755 --- a/src/jalview/jbgui/GDesktop.java +++ b/src/jalview/jbgui/GDesktop.java @@ -20,14 +20,7 @@ */ package jalview.jbgui; -import jalview.api.AlignmentViewPanel; -import jalview.bin.Jalview; -import jalview.io.FileFormatException; -import jalview.util.MessageManager; -import jalview.util.Platform; - import java.awt.FlowLayout; -import java.awt.Toolkit; import java.awt.event.ActionEvent; import java.awt.event.ActionListener; @@ -37,31 +30,22 @@ import javax.swing.JMenu; import javax.swing.JMenuBar; import javax.swing.JMenuItem; +import jalview.api.AlignmentViewPanel; +import jalview.bin.Cache; +import jalview.io.FileFormatException; +import jalview.util.MessageManager; +import jalview.util.Platform; + /** * DOCUMENT ME! * * @author $author$ * @version $Revision$ */ +@SuppressWarnings("serial") public class GDesktop extends JFrame { - public static javax.swing.JTabbedPane createTabbedPane() - { - // now just always return JTabbedPane - return new javax.swing.JTabbedPane(); -// // BH 2018 coercing jalview.jbgui.swing.JTabbedPane() for now -// if (/** @j2sNative false && */ -// true) -// { -// // Java -// return new javax.swing.JTabbedPane(); -// } -// // JavaScript -// return (javax.swing.JTabbedPane) (Object) new jalview.jbgui.swing.JTabbedPane(); - } - - protected static JMenu windowMenu = new JMenu(); JMenuBar desktopMenubar = new JMenuBar(); @@ -70,10 +54,6 @@ public class GDesktop extends JFrame JMenu HelpMenu = new JMenu(); - protected JMenu VamsasMenu = new JMenu(); - - protected JMenu VamsasStMenu = new JMenu(); - JMenuItem inputLocalFileMenuItem = new JMenuItem(); JMenuItem inputURLMenuItem = new JMenuItem(); @@ -94,20 +74,14 @@ public class GDesktop extends JFrame JMenuItem saveState = new JMenuItem(); + JMenuItem saveAsState = new JMenuItem(); + JMenuItem loadState = new JMenuItem(); JMenu inputMenu = new JMenu(); - protected JMenuItem vamsasStart = new JMenuItem(); - - protected JMenuItem vamsasImport = new JMenuItem(); - - protected JMenuItem vamsasSave = new JMenuItem(); - JMenuItem inputSequence = new JMenuItem(); - protected JMenuItem vamsasStop = new JMenuItem(); - JMenuItem closeAll = new JMenuItem(); JMenuItem raiseRelated = new JMenuItem(); @@ -143,11 +117,12 @@ public class GDesktop extends JFrame e.printStackTrace(); } - if (Platform.isWin()) //!Platform.isAMacAndNotJS()) + if (Platform.allowMnemonics()) { + // BH was !Platform.isAMacAndNotJS()) i.e. "JS or not Mac" + // but here we want just not a Mac, period, right? FileMenu.setMnemonic('F'); inputLocalFileMenuItem.setMnemonic('L'); - VamsasMenu.setMnemonic('V'); inputURLMenuItem.setMnemonic('U'); inputTextboxMenuItem.setMnemonic('C'); quit.setMnemonic('Q'); @@ -165,21 +140,37 @@ public class GDesktop extends JFrame */ private void jbInit() throws Exception { + /** + * APQHandlers sets handlers for About, Preferences and Quit actions + * peculiar to macOS's application menu. APQHandlers will check to see if a + * handler is supported before setting it. + */ + try + { + APQHandlers.setAPQHandlers(this); + } catch (Exception e) + { + System.out.println("Cannot set APQHandlers"); + // e.printStackTrace(); + } catch (Throwable t) + { + jalview.bin.Console + .warn("Error setting APQHandlers: " + t.toString()); + jalview.bin.Console.trace(Cache.getStackTraceString(t)); + } + setName("jalview-desktop"); FileMenu.setText(MessageManager.getString("action.file")); HelpMenu.setText(MessageManager.getString("action.help")); - VamsasMenu.setText("Vamsas"); - VamsasMenu.setToolTipText(MessageManager - .getString("label.share_data_vamsas_applications")); - VamsasStMenu.setText(MessageManager.getString("label.connect_to")); - VamsasStMenu.setToolTipText( - MessageManager.getString("label.join_existing_vamsas_session")); inputLocalFileMenuItem .setText(MessageManager.getString("label.load_tree_from_file")); - inputLocalFileMenuItem.setAccelerator( - javax.swing.KeyStroke.getKeyStroke(java.awt.event.KeyEvent.VK_O, - Toolkit.getDefaultToolkit().getMenuShortcutKeyMask(), - false)); + inputLocalFileMenuItem + .setAccelerator( + javax.swing.KeyStroke + .getKeyStroke(java.awt.event.KeyEvent.VK_O, + jalview.util.ShortcutKeyMaskExWrapper + .getMenuShortcutKeyMaskEx(), + false)); inputLocalFileMenuItem .addActionListener(new java.awt.event.ActionListener() { @@ -264,16 +255,16 @@ public class GDesktop extends JFrame @Override public void actionPerformed(ActionEvent e) { - saveState_actionPerformed(true); + saveState_actionPerformed(); } }); - JMenuItem saveAsJaxb = new JMenuItem("Save Project as JAXB"); - saveAsJaxb.addActionListener(new ActionListener() + saveAsState.setText(MessageManager.getString("action.save_project_as")); + saveAsState.addActionListener(new ActionListener() { @Override public void actionPerformed(ActionEvent e) { - saveState_actionPerformed(false); + saveAsState_actionPerformed(e); } }); loadState.setText(MessageManager.getString("action.load_project")); @@ -282,52 +273,10 @@ public class GDesktop extends JFrame @Override public void actionPerformed(ActionEvent e) { - loadState_actionPerformed(true); - } - }); - JMenuItem loadAsJaxb = new JMenuItem("Load Project as JAXB"); - loadAsJaxb.addActionListener(new ActionListener() - { - @Override - public void actionPerformed(ActionEvent e) - { - loadState_actionPerformed(false); + loadState_actionPerformed(); } }); inputMenu.setText(MessageManager.getString("label.input_alignment")); - vamsasStart - .setText(MessageManager.getString("label.new_vamsas_session")); - vamsasStart.setVisible(false); - vamsasStart.addActionListener(new ActionListener() - { - @Override - public void actionPerformed(ActionEvent e) - { - vamsasStart_actionPerformed(e); - } - }); - vamsasImport.setText( - MessageManager.getString("action.load_vamsas_session")); - vamsasImport.setVisible(false); - vamsasImport.addActionListener(new ActionListener() - { - @Override - public void actionPerformed(ActionEvent e) - { - vamsasImport_actionPerformed(e); - } - }); - vamsasSave.setText( - MessageManager.getString("action.save_vamsas_session")); - vamsasSave.setVisible(false); - vamsasSave.addActionListener(new ActionListener() - { - @Override - public void actionPerformed(ActionEvent e) - { - vamsasSave_actionPerformed(e); - } - }); inputSequence .setText(MessageManager.getString("action.fetch_sequences")); inputSequence.addActionListener(new ActionListener() @@ -338,17 +287,6 @@ public class GDesktop extends JFrame inputSequence_actionPerformed(e); } }); - vamsasStop - .setText(MessageManager.getString("label.stop_vamsas_session")); - vamsasStop.setVisible(false); - vamsasStop.addActionListener(new ActionListener() - { - @Override - public void actionPerformed(ActionEvent e) - { - vamsasStop_actionPerformed(e); - } - }); closeAll.setText(MessageManager.getString("action.close_all")); closeAll.addActionListener(new ActionListener() { @@ -453,38 +391,29 @@ public class GDesktop extends JFrame desktopMenubar.add(FileMenu); desktopMenubar.add(toolsMenu); - VamsasMenu.setVisible(false); - desktopMenubar.add(VamsasMenu); desktopMenubar.add(HelpMenu); desktopMenubar.add(windowMenu); FileMenu.add(inputMenu); FileMenu.add(inputSequence); FileMenu.addSeparator(); - FileMenu.add(saveAsJaxb); - FileMenu.add(loadAsJaxb); - if (!Jalview.isJS()) - { - FileMenu.add(saveState); - FileMenu.add(loadState); - } + // FileMenu.add(saveState); + FileMenu.add(saveAsState); + FileMenu.add(loadState); FileMenu.addSeparator(); FileMenu.add(quit); - HelpMenu.add(aboutMenuItem); + if (!APQHandlers.setAPQHandlers) + { + HelpMenu.add(aboutMenuItem); + } HelpMenu.add(documentationMenuItem); - VamsasMenu.add(VamsasStMenu); - VamsasStMenu.setVisible(false); - VamsasMenu.add(vamsasStart); - VamsasMenu.add(vamsasImport); - VamsasMenu.add(vamsasSave); - VamsasMenu.add(vamsasStop); - toolsMenu.add(preferences); - if (!Jalview.isJS()) + if (!APQHandlers.setAPQHandlers) { - toolsMenu.add(showMemusage); - toolsMenu.add(showConsole); + toolsMenu.add(preferences); } - if (!Jalview.isJS()) + if (!Platform.isJS()) { + toolsMenu.add(showMemusage); + toolsMenu.add(showConsole); toolsMenu.add(showNews); toolsMenu.add(garbageCollect); toolsMenu.add(groovyShell); @@ -586,6 +515,7 @@ public class GDesktop extends JFrame */ protected void quit() { + // System.out.println("********** GDesktop.quit()"); } /** @@ -608,7 +538,7 @@ public class GDesktop extends JFrame * @param e * DOCUMENT ME! */ - public void SaveState_actionPerformed(ActionEvent e) + protected void preferences_actionPerformed(ActionEvent e) { } @@ -618,17 +548,11 @@ public class GDesktop extends JFrame * @param e * DOCUMENT ME! */ - protected void preferences_actionPerformed(ActionEvent e) + public void saveState_actionPerformed() { } - /** - * DOCUMENT ME! - * - * @param e - * DOCUMENT ME! - */ - public void saveState_actionPerformed(boolean asCastor) + public void saveAsState_actionPerformed(ActionEvent e) { } @@ -638,7 +562,7 @@ public class GDesktop extends JFrame * @param e * DOCUMENT ME! */ - public void loadState_actionPerformed(boolean asCastor) + public void loadState_actionPerformed() { }