Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-06-14 Thread Andreas Heger
he bug >> [https://bugs.openjdk.java.net/browse/JDK-8255015](url) > > @andreas-heger Welcome to the `jfx` project. At a quick glance, the fix looks > promising. Have you tested this on Windows with Hi-DPI to make sure there is > no impact? Would you be able add an automated test

RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-06-09 Thread Andreas Heger
The inconsistent illumination happens on Macs with retina displays only if the 3D shape is placed in a SubScene. The light sources are located with wrong coordinates in sub scenes and this causes a different illumination. The wrong coordinates for the light sources come from the fact that the

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v2]

2021-06-09 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger

Withdrawn: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-06-09 Thread Andreas Heger
On Wed, 9 Jun 2021 18:02:30 GMT, Andreas Heger wrote: > The inconsistent illumination happens on Macs with retina displays only if > the 3D shape is placed in a SubScene. The light sources are located with > wrong coordinates in sub scenes and this causes a different ill

RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-06-09 Thread Andreas Heger
The inconsistent illumination happens on Macs with retina displays only if the 3D shape is placed in a SubScene. The light sources are located with wrong coordinates in sub scenes and this causes a different illumination. The wrong coordinates for the light sources come from the fact that the

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-06-09 Thread Andreas Heger
On Wed, 9 Jun 2021 18:46:16 GMT, Kevin Rushforth wrote: > @andreas-heger Welcome to the `jfx` project. At a quick glance, the fix looks > promising. Have you tested this on Windows with Hi-DPI to make sure there is > no impact? No, I only have got MacOS on retina / non retina and a n

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v2]

2021-09-14 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v3]

2021-09-16 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v4]

2021-09-20 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-09-22 Thread Andreas Heger
the calculated average does not differ from the >> excepted average color by a certain tolerance value. I'm not sure if I will >> manage to do this... but I will give it a try. > > @andreas-heger are you able to come up with an automated test for this bug? @kevinrushforth ye

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v10]

2021-10-24 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-10-24 Thread Andreas Heger
the calculated average does not differ from the >> excepted average color by a certain tolerance value. I'm not sure if I will >> manage to do this... but I will give it a try. > > @andreas-heger are you able to come up with an automated test for this bug? @kevinrushforth I've final

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v11]

2021-10-24 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v5]

2021-10-22 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v6]

2021-10-22 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v7]

2021-10-22 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v8]

2021-10-22 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

RFR: 8242544: CMD+ENTER key event crashes the application when invoked on dialog

2021-12-28 Thread Andreas Heger
This also solves [JDK-8205915 [macOS] Accelerator assigned to button in dialog fires menuItem in owning stage](https://bugs.openjdk.java.net/browse/JDK-8205915l). I must admit that I don't have 100% insight about what actually caused the problems and how the event flow completely works. In

Re: RFR: 8242544: CMD+ENTER key event crashes the application when invoked on dialog

2021-12-29 Thread Andreas Heger
On Wed, 29 Dec 2021 00:44:44 GMT, Andreas Heger wrote: > This also solves [JDK-8205915 [macOS] Accelerator assigned to button in > dialog fires menuItem in owning > stage](https://bugs.openjdk.java.net/browse/JDK-8205915l). > > I must admit that I don't have 100% insight abou

Re: RFR: 8276174: Build failure on MacBook Air (M1) in JFX component graphics [v2]

2021-11-18 Thread Andreas Heger
> By changing the value for the clang -arch parameter to "arm64", the jfx > project compiles on an apple silicon system. Are there any side effects which > I might be missing in this simple solution? Andreas Heger has updated the pull request incrementally with one addit

Re: RFR: 8276174: Build failure on MacBook Air (M1) in JFX component graphics

2021-11-18 Thread Andreas Heger
On Thu, 11 Nov 2021 13:37:37 GMT, Kevin Rushforth wrote: >> By changing the value for the clang -arch parameter to "arm64", the jfx >> project compiles on an apple silicon system. Are there any side effects >> which I might be missing in this simple solution? > > Did you build WebKit and media

Integrated: 8276174: JavaFX build fails on macOS aarch64

2021-11-22 Thread Andreas Heger
On Thu, 11 Nov 2021 10:46:26 GMT, Andreas Heger wrote: > By changing the value for the clang -arch parameter to "arm64", the jfx > project compiles on an apple silicon system. Are there any side effects which > I might be missing in this simple solution? This pull r

RFR: 8276174: Build failure on MacBook Air (M1) in JFX component graphics

2021-11-11 Thread Andreas Heger
By changing the value for the clang -arch parameter to "arm64", the jfx project compiles on an apple silicon system. Are there any side effects which I might be missing in this simple solution? - Commit messages: - Merge branch 'openjdk:master' into fix-8276174 - 8276174: Use

Integrated: 8255015: Inconsistent illumination of 3D shape by PointLight

2021-10-29 Thread Andreas Heger
On Wed, 9 Jun 2021 18:22:31 GMT, Andreas Heger wrote: > The inconsistent illumination happens on Macs with retina displays only if > the 3D shape is placed in a SubScene. The light sources are located with > wrong coordinates in sub scenes and this causes a different illumination. The

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v12]

2021-10-28 Thread Andreas Heger
On Wed, 27 Oct 2021 09:36:20 GMT, Ambarish Rapte wrote: >> Andreas Heger has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8255015: testScene variable must be volatile and new line at the end of >> the f

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v13]

2021-10-28 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v11]

2021-10-26 Thread Andreas Heger
On Mon, 25 Oct 2021 23:50:09 GMT, Kevin Rushforth wrote: >> Andreas Heger has updated the pull request with a new target base due to a >> merge or a rebase. The incremental webrev excludes the unrelated changes >> brought in by the merge/rebase. The pull request cont

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v12]

2021-10-26 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h

Re: RFR: 8255015: Inconsistent illumination of 3D shape by PointLight [v9]

2021-10-22 Thread Andreas Heger
rom the fact that the retina > pixel scale factors are not used in a SubScene. > > With this pull request, the retina pixel scale factors will be also used in > SubScenes and this should resolve the bug > [https://bugs.openjdk.java.net/browse/JDK-8255015](url) Andreas Heger h