Thank you Sergey for the review. Denis, Please let me know if you have any comments on the changes.
Thanks and regards, Shashi -----Original Message----- From: Sergey Bylokhov Sent: Thursday, April 12, 2018 3:36 AM To: Denis Fokin <denis.fo...@gmail.com> Cc: Hendrik Schreiber <h...@tagtraum.com>; Shashidhara Veerabhadraiah <shashidhara.veerabhadra...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: <AWT Dev> [11] JDK-8181910: [macos] Support dark title bars on macOS Thank you, the fix looks fine. On 11/04/2018 07:44, Denis Fokin wrote: > Hi Sergey, > > can I just approve any modifications and usage of the patch? I am > absolutely ok with pushing or improving the fix by Shashidhara. > > __ > Thank you, > Denis > > On Tue, Apr 3, 2018 at 10:50 PM Sergey Bylokhov > <sergey.bylok...@oracle.com <mailto:sergey.bylok...@oracle.com>> wrote: > > Yes, this fix can be contributed by someone from the JB. > > On 03/04/2018 06:48, Hendrik Schreiber wrote: > > Hey, > > > > I was wondering how this is going. Are you guys still stuck, > waiting for Denis to help out? > > > > -hendrik > > > > > >> On Mar 30, 2018, at 07:32, shashidhara.veerabhadra...@oracle.com > <mailto:shashidhara.veerabhadra...@oracle.com> wrote: > >> > >> Sure. I did had a confusion to put for review though but did not > know what to do and felt not to keep waiting. Thanks for the > direction Sergey. > >> > >> Thanks and regards, > >> Shashi > >> > >> > >> On 29/03/18 7:45 PM, Sergey Bylokhov wrote: > >>> Unfortunately we cannot accept the patch which were suggested > in the description of the bug. Initially it was implemented in > JetBrains JRE. We can accept it if someone from the JetBrains will > contribute it. > >>> > >>> Maybe Denis who is author of the fix can help? > >>> > >>> On 28/03/2018 02:29, Shashidhara Veerabhadraiah wrote: > >>>> Hi All, Please review an enhancement for the below bug: > >>>> > >>>> Enhancement: https://bugs.openjdk.java.net/browse/JDK-8181910 > >>>> > >>>> Webrev: > http://cr.openjdk.java.net/~sveerabhadra/8181910/webrev.00/ > >>>> > >>>> This utilizes the NSAppearance to set the appearance theme to > dark or light. > >>>> > >>>> Thanks and regards, > >>>> > >>>> Shashi > >>>> > >>> > >>> > >> > > > > > -- > Best regards, Sergey. > -- Best regards, Sergey.
