Commit on focus loss in ListView, TreeView, TableView, and TreeTableView

2017-05-04 Thread Jonathan Giles
Hi all, In the next week I have two topics to discuss, both with similar names but related to quite different functionality. The topic for today is adding the necessary API to the JavaFX UI control Cell class to enable support for 'commit on focus loss', that is, the ability to automatically

Re: javafx.scene.shape.Path (memory) inefficient PathElements

2017-05-04 Thread Jim Graham
Hi Tom, Those look like good suggestions. I would file bugs in JBS and create them separately: - Bug for lazy property creation in path elements - Feature request for lower-memory paths Did you benchmark how much the lazy properties, on their own, would save your application?

javafx.scene.shape.Path (memory) inefficient PathElements

2017-05-04 Thread Tom Schindl
Hi, We are currently working on a PDF-Rendering library in JavaFX and we need to draw glyphs using the JavaFX Path API (there are multiple reasons why we don't use the Text-Node and or Canvas). When drawing a page full of Text this means that we have a Path-Object with gazillions of MoveTo and Cu

API docs for javafx.* modules are now included with the JDK 9 API docs

2017-05-04 Thread Kevin Rushforth
Some of you may have noticed already that the API docs for javafx.* modules are now included as part of the JDK 9 API docs on java.net: http://download.java.net/java/jdk9/docs/api/ The search box also works as expected to find JavaFX classes as well an any classes in the rest of the JDK. Hopef

Re: [9] Review request: 8177566: FX user module gets IllegalAccessException from sun.reflect.misc.Trampoline

2017-05-04 Thread Kevin Rushforth
This seems like something that could be useful, although at this point in the release we would more likely do it for JDK 10. I do note that including the class that made the illegal access is generally a good idea when that class is attempting the access on its own behalf. For frameworks such a

Re: [9] Review request: 8177566: FX user module gets IllegalAccessException from sun.reflect.misc.Trampoline

2017-05-04 Thread Gregg Wonderly
If there is not already such an exception, it would seem like a good idea to have an exception that formats such a message from constructor parameters providing the details so that it’s the same everywhere, and so that it can be changed in once place if needed. Gregg > On May 3, 2017, at 9:48

[10] Review request : JDK-8089984 : [Accessibility,CSS] Progress Indicator's Label is not visible in High Contrast

2017-05-04 Thread Ajit Ghaisas
Hi Jonathan, Request you to review following fix : Bug : https://bugs.openjdk.java.net/browse/JDK-8089984 Fix : http://cr.openjdk.java.net/~aghaisas/fx/8089984/webrev.0/ Regards, Ajit