Re: [9] Review request for 8166683: On macOS (Mac OS X) getting a ScreenMenuBar when not running "com.apple.laf.AquaLookAndFeel"

2016-12-07 Thread Philip Race
+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.

Re: [9] Review request for 8166683: On macOS (Mac OS X) getting a ScreenMenuBar when not running "com.apple.laf.AquaLookAndFeel"

2016-12-07 Thread Sergey Bylokhov
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

Re: [9] Review request for 8165428: Security Warning dialog should be always on the top when multiple applets with APPLICATION_MODAL dialog launched in a browser

2016-12-07 Thread dmitry markov
Thank you very much, Sergey! Looking for the second +1 from someone else. Thanks, Dmitry On 07/12/2016 19:25, Sergey Bylokhov wrote: Looks fine. 7 дек. 2016 г., в 2:24, Dmitry Markov > написал(а): Hi Sergey, I agree, it is not necessary to increase the toolki

Re: [9] Review request for 8165428: Security Warning dialog should be always on the top when multiple applets with APPLICATION_MODAL dialog launched in a browser

2016-12-07 Thread Sergey Bylokhov
Looks fine. > 7 дек. 2016 г., в 2:24, Dmitry Markov написал(а): > > Hi Sergey, > > I agree, it is not necessary to increase the toolkit counter here. It is a > copy-paste error. I am sorry about that. Please find the updated webrev here: > http://cr.openjdk.java.net/~dmarkov/8165428/webrev.02

Re: [9] Review request for 8165428: Security Warning dialog should be always on the top when multiple applets with APPLICATION_MODAL dialog launched in a browser

2016-12-07 Thread Dmitry Markov
Hi Sergey, I agree, it is not necessary to increase the toolkit counter here. It is a copy-paste error. I am sorry about that. Please find the updated webrev here: http://cr.openjdk.java.net/~dmarkov/8165428/webrev.02/ Thanks, Dmitry > On 07 Dec 2016, at 03:40, Sergey Bylokhov wrote: > > Thi