RE: Usage of Toolkit firePulse

2015-09-24 Thread Fisher, Robert
I was naively thinking something like: 1. Make small change to canvas 2. Fire pulse 3. Make next small change to canvas 4. Fire pulse 5. Etc.. But I was actually also unaware of this firePulse method until this morning (and I couldn't have used it anyway since it's not public API).

Value for region size in CSS Reference Guide

2015-09-24 Thread Jörg Wille
After VW's cheating, I try to rehabilitate the reputation of german engineers and prove that most Germans do there work very precisely or even pedantic ;-) Can it be that there is a mistake in the CSS Reference Guide (

Re: Usage of Toolkit firePulse

2015-09-24 Thread Ryan Jaeb
I use firePulse() in GUI test code. I'm using TestFX (v3) for testing. Is there a way to ensure model updates are reflected in the scene without using firePulse()? Consider something like this: ... // TestFX starts us on the main thread. Maybe I'm using it wrong? CountDownLatch latch = new

Re: Usage of Toolkit firePulse

2015-09-24 Thread Johan Vos
I have to admit I use firePulse() occasionally but only when for some reasons a pulse isn't triggered while I expected it. That is a very bad usecase for having this as an API, as I would rather have to investigate why changes to the scenegraph don't result in a pulse. If firePulse() is a public

RE: Usage of Toolkit firePulse

2015-09-24 Thread Fisher, Robert
Well what about these use cases? 1. You have a ListView whose cells contain some graphics and are not lightning-fast to render. You also have a TextField at the top that filters the ListView content, i.e. a standard filtered list. When the user types something in the TextField you may want to

Re: Usage of Toolkit firePulse

2015-09-24 Thread Dr. Michael Paus
Hi, I wasn't aware of this Toolkit method when I wrote the mail you are referring to. Can you or anybody else explain what this method exactly does. It sounds indeed as if I could solve some problems with it although I am not sure yet and of course only if Jonathan does not block it in the

Backport for [JDK-8131151] Pressing ESC in a nested alert closes self and parent alert

2015-09-24 Thread Werner Lehmann
Hi, since there seems to be an opportunity to have fixes backported to 8, I'd like to ask for a backport of the fix for [JDK-8131151] Pressing ESC in a nested alert closes self and parent alert https://bugs.openjdk.java.net/browse/JDK-8131151 Looks as if the fix is pretty simple. Without

RE: Usage of Toolkit firePulse

2015-09-24 Thread Fisher, Robert
I think it would be great to have in the public API. It looks like it would allow you to spread large UI updates out over several pulses in a well-defined way. See also this post from a month or so ago: > Hi, > I want to do some performance tuning of a JavaFX application of mine but > before

Re: Usage of Toolkit firePulse

2015-09-24 Thread Scott Palmer
For some of these use cases I wonder if an AnimationTimer could be used to handle spreading work out. E.g in the case of rendering to a canvas across multiple pulses, do a bit at a time in the AnimationTimer’s handle() method. Scott > On Sep 24, 2015, at 5:53 AM, Fisher, Robert

Re: Usage of Toolkit firePulse

2015-09-24 Thread Dr. Michael Paus
I also thought about using an AnimationTimer but I see a problem with the load balancing there. If you do too much in the handle() method, then the pulses will be delayed but if you do not do enough, your canvas update is unnecessarily stretched and you are wasting time. You would have a

RE: Usage of Toolkit firePulse

2015-09-24 Thread Fisher, Robert
Thanks, that's an interesting point. I'll look into it. -Original Message- From: Scott Palmer [mailto:swpal...@gmail.com] Sent: Donnerstag, 24. September 2015 15:13 To: Fisher, Robert Cc: Dr. Michael Paus; openjfx-dev@openjdk.java.net Subject: Re: Usage of Toolkit firePulse For some of

Re: Usage of Toolkit firePulse

2015-09-24 Thread Kevin Rushforth
The problem with fireaPulse is that its semantics are somewhat ill-defined. It doesn't really run a full pulse, rather it just calls the various pulse listeners. So any animation will not be run and no rendering will be done (although any updates to the scene graph would trigger a future full

Re: Usage of Toolkit firePulse

2015-09-24 Thread Kevin Rushforth
This wouldn't do what you expect. :) -- Kevin Fisher, Robert wrote: I was naively thinking something like: 1. Make small change to canvas 2. Fire pulse 3. Make next small change to canvas 4. Fire pulse 5. Etc.. But I was actually also unaware of this firePulse method until this morning

RE: Usage of Toolkit firePulse

2015-09-24 Thread Fisher, Robert
Ok. Thanks for the info. From: Kevin Rushforth [mailto:kevin.rushfo...@oracle.com] Sent: Donnerstag, 24. September 2015 17:12 To: Fisher, Robert Cc: Dr. Michael Paus; openjfx-dev@openjdk.java.net Subject: Re: Usage of Toolkit firePulse This wouldn't do what you expect. :) -- Kevin Fisher,

Re: Usage of Toolkit firePulse

2015-09-24 Thread Kevin Rushforth
This is basically how we use it for our unit tests, and is so far the only use case I've see where the existing semantics are useful. Given that unit tests still need to dip into non-public API to test various non-public API, force layout, force a pulse, use glass Robot, etc., we wouldn't make

Re: Usage of Toolkit firePulse

2015-09-24 Thread Kevin Rushforth
Yes, I think this might be a better approach. -- Kevin Scott Palmer wrote: For some of these use cases I wonder if an AnimationTimer could be used to handle spreading work out. E.g in the case of rendering to a canvas across multiple pulses, do a bit at a time in the AnimationTimer’s

Re: Value for region size in CSS Reference Guide

2015-09-24 Thread Kevin Rushforth
This does seem like a bug in the docs, given that the implementation uses SizeConverter when processing those fields. You can file a bug at: http://bugs.java.com/ See https://wiki.openjdk.java.net/display/OpenJFX/Submitting+a+Bug+Report for more information about filing bugs. Thanks. --

Re: Usage of Toolkit firePulse

2015-09-24 Thread Tomas Mikula
I think using Platform.runLater to schedule chunks of work should work. However, dumping all chunks to the event queue at once will likely delay the next pulse and event processing. Using the continuation style, where each chunk schedules the next one, should work IMO. interface Continuation

Re: Usage of Toolkit firePulse

2015-09-24 Thread Dr. Michael Paus
I have written a little test program to evaluate the various strategies which have been discussed here. You can download it via this link: The program creates a large canvas inside a scroll pane and fills it with some

Re: Usage of Toolkit firePulse

2015-09-24 Thread Tomas Mikula
Hi Michael, attached see your original file updated with the continuation-style solution. Regards, Tomas On Thu, Sep 24, 2015 at 2:44 PM, Dr. Michael Paus wrote: > I have written a little test program to evaluate the various strategies > which have > been discussed here. You

JEP 275: Modular Java Application Packaging

2015-09-24 Thread mark . reinhold
New JEP Candidate: http://openjdk.java.net/jeps/275 - Mark