Re: RFR: 8242523: Update the animation and clip envelope classes

2020-05-13 Thread Ambarish Rapte
On Wed, 13 May 2020 12:29:03 GMT, Kevin Rushforth wrote: >> I agree that these methods are better suited there, but I'm not sure the >> same epsilon value will be suitable for other >> places that will want to use these. That value is somewhat arbitrary anyway >> I think. > > Either is fine

Re: RFR: 8242523: Update the animation and clip envelope classes

2020-05-13 Thread Ambarish Rapte
On Wed, 13 May 2020 00:38:03 GMT, Nir Lisker wrote: >> modules/javafx.graphics/src/main/java/com/sun/scenario/animation/shared/MultiLoopClipEnvelope.java >> line 48: >> >>> 47: >>> 48: protected boolean autoReverse() { >>> 49: return autoReverse; >> >> I would suggest the name to

Re: [Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Ambarish Rapte
On Wed, 13 May 2020 11:20:25 GMT, Jeanette Winzenburg wrote: >> modules/javafx.controls/src/test/java/test/javafx/scene/control/skin/SkinDisposeContractTest.java >> line 159: >> >>> 158: {TreeTableView.class, }, >>> 159: {TreeView.class, }, >>> 160: }; >> >>

Re: [Rev 04] RFR: 8217472: Add attenuation for PointLight

2020-05-13 Thread Kevin Rushforth
On Wed, 13 May 2020 23:42:40 GMT, Nir Lisker wrote: >> If this were an even remotely representative use case, then no, the >> performance hit would not be OK. The test was >> designed as an artificial "worst-case" stress test: a single mesh with a >> large number of very large (window-sized)

Re: [Rev 04] RFR: 8217472: Add attenuation for PointLight

2020-05-13 Thread Nir Lisker
On Sat, 25 Apr 2020 17:07:21 GMT, Kevin Rushforth wrote: > We should make sure that we aren't seeing any significant performance drop > when rendering spheres (at a couple > different tessellation levels) or boxes. I missed this. Do you mean that the test should create a mesh of a sphere

Re: RFR: 8242523: Update the animation and clip envelope classes

2020-05-13 Thread Nir Lisker
On Thu, 7 May 2020 08:50:23 GMT, Ambarish Rapte wrote: >> Mostly refactoring in preparation of the upcoming fixes. The changes might >> look like a lot, but it's mostly rearranging >> of methods. Summery of changes: >> ### Animation >> * Added `isNearZero` and `areNearEqual` methods that deal

Re: RFR: 8242523: Update the animation and clip envelope classes

2020-05-13 Thread Kevin Rushforth
On Wed, 13 May 2020 00:17:17 GMT, Nir Lisker wrote: >> modules/javafx.graphics/src/main/java/com/sun/scenario/animation/shared/ClipEnvelope.java >> line 46: >> >>> 45: */ >>> 46: public abstract class ClipEnvelope { >>> 47: >> >> I think the removal of line 46 was unintended change. > > I

Re: RFR: 8238080: FXMLLoader: if script engines implement javax.script.Compilable compile scripts

2020-05-13 Thread Rony G . Flatscher
On Wed, 13 May 2020 09:22:20 GMT, Rony G. Flatscher wrote: >> Doc addition for the CSR: >> = >> The following text was added to the [Introduction to >>

Re: [Rev 02] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Jeanette Winzenburg
> some skins have not been guarding themselves against multiple calls to > dispose (see issue for details) > > Fixed by backing out off dispose if skinnable is null. Added test > (parameterized in control class) for all controls in > the controls package. Those that failed for the misbehaving

Re: [Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Jeanette Winzenburg
On Wed, 13 May 2020 10:09:40 GMT, Ambarish Rapte wrote: >> Jeanette Winzenburg has updated the pull request incrementally with one >> additional commit since the last revision: >> >> cleanup >> >> - corrected incorrect bug id in commented >> - removed unrelated test change from

Re: [Rev 01] RFR: 8244418: MenuBar: IOOB exception on requestFocus on empty bar

2020-05-13 Thread Jeanette Winzenburg
On Tue, 12 May 2020 12:21:14 GMT, Ajit Ghaisas wrote: >> modules/javafx.controls/src/main/java/javafx/scene/control/skin/MenuBarSkin.java >> line 485: >> >>> 484: } >>> 485: >>> 486: if (focusedMenu != null && focusedMenuIndex != -1) { >> >> don't quite understand why this

Re: [Integrated] RFR: 8244647: Wrong first layout pass of Scrollbar controls on touch supported devices

2020-05-13 Thread Jose Pereda
On Fri, 8 May 2020 11:25:03 GMT, Jose Pereda wrote: > There is a visual glitch when the scrollbar controls are laid out on touch > enabled devices. > > The first time they are laid out in the wrong location (20 px from right or > bottom), while the next passes are correct > (8 px from right

Re: [Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Ambarish Rapte
On Mon, 11 May 2020 04:51:51 GMT, Ambarish Rapte wrote: >> Jeanette Winzenburg has updated the pull request incrementally with one >> additional commit since the last revision: >> >> cleanup >> >> - corrected incorrect bug id in commented >> - removed unrelated test change from

Re: [Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Ambarish Rapte
On Wed, 6 May 2020 11:46:27 GMT, Jeanette Winzenburg wrote: >> some skins have not been guarding themselves against multiple calls to >> dispose (see issue for details) >> >> Fixed by backing out off dispose if skinnable is null. Added test >> (parameterized in control class) for all

Re: [Rev 01] RFR: 8244647: Wrong first layout pass of Scrollbar controls on touch supported devices

2020-05-13 Thread Ajit Ghaisas
On Wed, 13 May 2020 09:38:23 GMT, Jose Pereda wrote: >> There is a visual glitch when the scrollbar controls are laid out on touch >> enabled devices. >> >> The first time they are laid out in the wrong location (20 px from right or >> bottom), while the next passes are correct >> (8 px from

Re: [Rev 01] RFR: 8244112: Skin implementations: must not violate contract of dispose

2020-05-13 Thread Ajit Ghaisas
On Wed, 6 May 2020 11:46:27 GMT, Jeanette Winzenburg wrote: >> some skins have not been guarding themselves against multiple calls to >> dispose (see issue for details) >> >> Fixed by backing out off dispose if skinnable is null. Added test >> (parameterized in control class) for all

Re: [Rev 01] RFR: 8244647: Wrong first layout pass of Scrollbar controls on touch supported devices

2020-05-13 Thread Jose Pereda
On Wed, 13 May 2020 09:14:02 GMT, Ajit Ghaisas wrote: >> Jose Pereda has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Apply code formatting > > modules/javafx.controls/src/main/java/javafx/scene/control/skin/VirtualFlow.java > line

Re: [Rev 01] RFR: 8244647: Wrong first layout pass of Scrollbar controls on touch supported devices

2020-05-13 Thread Jose Pereda
> There is a visual glitch when the scrollbar controls are laid out on touch > enabled devices. > > The first time they are laid out in the wrong location (20 px from right or > bottom), while the next passes are correct > (8 px from right or bottom). > The reason for this glitch is the use of

Re: RFR: 8238080: FXMLLoader: if script engines implement javax.script.Compilable compile scripts

2020-05-13 Thread Rony G . Flatscher
On Tue, 12 May 2020 17:33:06 GMT, Rony G. Flatscher wrote: >> Suggested CSR: >> >> Summary >> === >> Have javafx.fxml.FXMLLoader compile FXML scripts before evaluating them, if >> the script engine >> implements the javax.script.Compilable interface to speed up execution. In >> case

Re: RFR: 8244647: Wrong first layout pass of Scrollbar controls on touch supported devices

2020-05-13 Thread Ajit Ghaisas
On Fri, 8 May 2020 11:25:03 GMT, Jose Pereda wrote: > There is a visual glitch when the scrollbar controls are laid out on touch > enabled devices. > > The first time they are laid out in the wrong location (20 px from right or > bottom), while the next passes are correct > (8 px from right

Re: HID multitouch display support in JavaFX

2020-05-13 Thread Michael Paus
Having proper and consistent Multitouch support on all platforms is certainly an interesting goal, especially if your development is targeting a mobile platform. If one could test the input behavior already on the desktop, this would be an enormous time saver because the turn-around times

Re: [Rev 01] RFR: 8238080: FXMLLoader: if script engines implement javax.script.Compilable compile scripts

2020-05-13 Thread Rony G . Flatscher
> This WIP adds the ability for a fallback in case compilation of scripts > fails, in which case a warning gets issued > about this fact and evaluation of the script will be done without > compilation. Because of the fallback scripts get > compiled with this version by default. It extends PR 187

Re: TableView help

2020-05-13 Thread Dirk Lemmermann
Hi John, the model class used by your TableView is called “SimpleImmutableEntry”. The middle part of this name should be enough to tell you what the problem is :-) Solution: create your own custom model class and provide setters and getters for “key” and “value”. Then the TableView will write

Re: HID multitouch display support in JavaFX

2020-05-13 Thread jfx user2
Nobody is biting... so is anyone else out there interested in using HID touchscreens with JavaFX on OSX (or any other plaform) without using a third party driver? On Fri, May 8, 2020 at 4:18 AM jfx user2 wrote: > Multitouch display support JavaFX on Windows, iOS, and Android seems >

Re: com.sun.javafx vs javafx (with example/request)

2020-05-13 Thread jfx user2
Turns out that I partially misunderstood the use of Path2D in GraphicsContext. I had been using appendSVGPath for nearly everything and within a single beginPath/closePath. Under those circumstances converting GraphicsContext.path from Path2D to Path (using code from