Re: [10] Review request for 8182638: [macosx] Active modal dialog is hidden by another non-active one

2017-10-09 Thread Dmitry Markov
Hi Semyon, Looks good to me. Thanks, Dmitry > On 6 Oct 2017, at 20:47, Semyon Sadetsky wrote: > > Hello, > > Please review fix for JDK10: > > bug: https://bugs.openjdk.java.net/browse/JDK-8182638 > > webrev:

Re: [10] Review request for 8182638: [macosx] Active modal dialog is hidden by another non-active one

2017-10-08 Thread Alexander Zvegintsev
Looks good to me, BTW in the test DialogListener can be replaced with WindowAdapter Thanks, Alexander. On 07/10/2017 01:17, Semyon Sadetsky wrote: Hello, Please review fix for JDK10: bug: https://bugs.openjdk.java.net/browse/JDK-8182638 webrev:

[10] Review request for 8182638: [macosx] Active modal dialog is hidden by another non-active one

2017-10-06 Thread Semyon Sadetsky
Hello, Please review fix for JDK10: bug: https://bugs.openjdk.java.net/browse/JDK-8182638 webrev: http://cr.openjdk.java.net/~ssadetsky/8182638/webrev.00/ Since 8169589 sibling windows are ordered above the parent. It requires for the new active child windows to be raised above its siblings