Menu

#598 About item under Application name to work

SQuirreL
open
nobody
None
medium
2021-04-11
2021-02-15
Heklaz
No

A small patch to make the "About" item under Application name to work is attached.

Does require Java 9 or newer.

(I also have some changes to make squirrelsql compile with Java 11 if you're interested, all in AutoCorrectController, compiles but not really tested)

1 Attachments

Related

Feature Requests: #598

Discussion

  • Heklaz

    Heklaz - 2021-02-15

    I completely forgot; it is was macos.

     
  • Stanimir Stamenkov

    I also have some changes to make squirrelsql compile with Java 11 if you're interested...

    I guess it would be pretty much the same as:

    • [91e3c1] Build: Allow building with Java 9+

    part of merge-request #57. There I also suggest multi-release JAR packaging compatible with Java8/9+, but Gerd is not convinced so far.

    As you're on Mac, I guess/hope you're using a retina display and I wonder if you're interested and you could give a try of merge-request #57?

     

    Last edit: Stanimir Stamenkov 2021-02-15
  • Heklaz

    Heklaz - 2021-02-16

    I could probably could do that.

    May need a little help here, how do I fetch the merge-request from commandline git?

    It will be a few days before I can do more with this.

     
    • Stanimir Stamenkov

      You could clone my fork directly:

      git clone https://git.code.sf.net/u/stanio/squirrel-sql
      

      or you could add it as a remote to existing clone:

      git remote add -f stanio https://git.code.sf.net/u/stanio/squirrel-sql
      

      In both cases:

      git checkout multires-icons
      

      then cd sql12 and build using ant. You could try running directly from the working copy using ant runApp, or you could perform install from the output/izPackInstallJars.

       
  • Heklaz

    Heklaz - 2021-02-22

    Hi,

    tried that and the application runs.

    What should I look for here to be sure I get useful results?

     
    • Stanimir Stamenkov

      Hi Heklaz. Thank you for trying it out! I'm interest to find out if you're seeing any improvement (none, somewhat, significant) in the quality of the icons, working on a hidpi/retina screen. I guess it is better to continue this exchange over in merge-request #57.

       
  • Gerd Wagner

    Gerd Wagner - 2021-02-22

    I'm able to reproduce your problem: When SQuirreL is installed on a Mac using the the Mac Installer one can't open the SQuirreL's About message box.
    I also see that your patch works. But I'm sorry SQuirreL is still based in Java 1.8 and I don't consider the feature relevant enough to switch to Java 11.
    As a workaround you may use the standard installer. This will make the About box available although for the cost of loosing some Mac integration features.
    Gerd

     
  • Gerd Wagner

    Gerd Wagner - 2021-02-23

    Just committed a fix which ensures that all menus are available on Mac OS. The fix will be available in future snapshots and versions.

     
  • Gerd Wagner

    Gerd Wagner - 2021-03-09

    I agree it doesn't show under the menu named "SQuirreLSQL". But its available under the "Help" menu. I think that's not perfect but rather alright.

    I noticed the exit command, ⌘-Q problem, too. This one I consider quite severe because it passes saving preferences on close. By now I think that after the 4.2.0 release we should switch to Java 11 and make some use of the desktop integration features.

     
  • Gerd Wagner

    Gerd Wagner - 2021-04-11

    The new snapshot
    https://sourceforge.net/projects/squirrel-sql/files/3-snapshots/snapshot-20210411_2107/

    requires Java 11 and makes Apple's application close command "Command key"+Q work the same way Ctrl+Q does. It also properly supports the Apple specific About-Command.

     

Log in to post a comment.