> This pull request fixes the system menu bar on Mac when combining windows of 
> Swing and JavaFX.
> 
> The first issue was to get the native menu bar working simultaneously on 
> Swing and JavaFX, which was done by just returning always true inside the 
> supportsSystemMenu method.
> 
> The second issue was to remove all system menu items installed by a swing 
> window. This was fixed by checking the system menu bar every time an item is 
> inserted or removed and removing all menu items that are not owned by JavaFX. 
> This check is done on every insert and remove as JavaFX does not have a clear 
> method inside the MenuBarDelegate class that could be called every time the 
> window gets the focus.
> 
> I tested the fix with two Swing and two JavaFX windows that are run inside 
> the same application and it worked without any errors, but on further testing 
> I noticed some issues with the menu bar. I am currently writing the test and 
> fixes for it.
> 
> Co-Author: @FlorianKirmaier

Pabulaner IV has updated the pull request incrementally with one additional 
commit since the last revision:

  8359108: Mac - When Swing starts First, native application menu doesn't work 
for JavaFX

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/1835/files
  - new: https://git.openjdk.org/jfx/pull/1835/files/fc230981..6f5ec818

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=1835&range=03
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=1835&range=02-03

  Stats: 11 lines in 1 file changed: 0 ins; 0 del; 11 mod
  Patch: https://git.openjdk.org/jfx/pull/1835.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/1835/head:pull/1835

PR: https://git.openjdk.org/jfx/pull/1835

Reply via email to