+1
-phil.
On 12/7/16, 12:51 PM, Sergey Bylokhov wrote:
Looks fine.
5 дек. 2016 г., в 22:52, Alexander Zvegintsev
написал(а):
Actually there is no need in this property, this behavior can be disabled for
other L&F by setting apple.laf.useScreenMenuBar property to false.
http://cr.openjdk.
Looks fine.
> 5 дек. 2016 г., в 22:52, Alexander Zvegintsev
> написал(а):
>
> Actually there is no need in this property, this behavior can be disabled for
>
> other L&F by setting apple.laf.useScreenMenuBar property to false.
>
> http://cr.openjdk.java.net/~azvegint/jdk/9/8166683/03/
>
> th
Actually there is no need in this property, this behavior can be
disabled for
other L&F by setting apple.laf.useScreenMenuBar property to false.
http://cr.openjdk.java.net/~azvegint/jdk/9/8166683/03/
the fix is also reworked to remove mac specific stuff from shared code.
Thanks,
Alexander.
O
I don't find any modern jdk9 prefix convention for such property, so
I've named it "jdk.swing.disableForcedGlobalMenuBar"
http://cr.openjdk.java.net/~azvegint/jdk/9/8166683/02/
Thanks,
Alexander.
On 11/28/16 9:05 PM, Sergey Bylokhov wrote:
Looks fine, but here is some of my thoughts:
Since w
Looks fine, but here is some of my thoughts:
Since we tries to provide some kind of public API, I suggest to double
check the solution again. In fact we tried to provide a support of the
global menu on osx for all our L&Fs.
- Is it necessary to reference the Aqua from the shared code? in
varia
Looks good.
--Semyon
On 11/15/2016 5:39 PM, Alexander Zvegintsev wrote:
Hi Sergey,
I've not found casting issues, but I've found the issue when previous
fix does not
treat dynamically changed "apple.laf.useScreenMenuBar" property
correctly. (e.g. ScreenMenuBarInputTwice test fails).
So
Hi Sergey,
I've not found casting issues, but I've found the issue when previous
fix does not
treat dynamically changed "apple.laf.useScreenMenuBar" property
correctly. (e.g. ScreenMenuBarInputTwice test fails).
So please see the updated changeset:
http://cr.openjdk.java.net/~azvegint/jdk/
Hi, Alexander.
Did you run the tests on non-Aqua l&f? I assume that we can have a
places in other l&f where we try to cast the MenuBarUI to some specific
UI delegate.
On 09.11.16 16:58, Alexander Zvegintsev wrote:
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/81666
Hello,
please review the fix
http://cr.openjdk.java.net/~azvegint/jdk/9/8166683/00/
for the issue
https://bugs.openjdk.java.net/browse/JDK-8166683
This fix adds support for ScreenMenuBar for L&F's other than Aqua.
With this fix it is enabled by default if apple.laf.useScreenMenuBar
property