hg: openjfx/8u-dev/rt: 2 new changesets
Changeset: e391dce8d6d7 Author:Martin Sladecek martin.slade...@oracle.com Date: 2014-04-23 08:15 +0200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/e391dce8d6d7 RT-29507 [Chart] axis rotate can be lost. Reviewed by: jgiles ! modules/controls/src/main/java/javafx/scene/chart/Axis.java ! modules/controls/src/main/java/javafx/scene/chart/CategoryAxis.java Changeset: 0e59417d5951 Author:Martin Sladecek martin.slade...@oracle.com Date: 2014-04-23 08:16 +0200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/0e59417d5951 RT-36766 Right most Slider Axis label sometimes invisible. Reviewed by: jgiles ! modules/controls/src/main/java/javafx/scene/chart/ValueAxis.java
hg: openjfx/8u-dev/rt: RT-36801 : Monocle input test fails and then hangs on Windows and Mac
Changeset: 7ecd4c4dcf82 Author:Rafi Tayar rafi.ta...@oracle.com Date: 2014-04-23 18:39 +0300 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/7ecd4c4dcf82 RT-36801 : Monocle input test fails and then hangs on Windows and Mac Reviewed-by : dblaukopf. ! tests/system/src/test/java/com/sun/glass/ui/monocle/input/CreateDeviceTest.java ! tests/system/src/test/java/com/sun/glass/ui/monocle/input/SingleTouchTest.java ! tests/system/src/test/java/com/sun/glass/ui/monocle/input/TestApplication.java ! tests/system/src/test/java/com/sun/glass/ui/monocle/input/TestLog.java
Re: Should Canvas have been Image subclass instead of Node?
That's a nice solution - though in real world code I worry that the endDrawing snapshotting would be triggered more often than necessary. I still think there is no compelling reason for Canvas to be a Node, and there is a significant drawback if you want to reuse the same Canvas many times in your scene graph. I think it's appropriate to think of the basic Canvas functionality as a Procedural Image - it is used to create an Image from code instead of a file. Just because the most common case might be to stuff it in an ImageView, doesn't mean that it should be a Node for convenience. If you agree that the Image class shouldn't be a Node, you should feel the same way about Canvas. I also think this would be easy to fix in a backward compatible way. A future release could simply add a CanvasImage class which had a GraphicsContext - then Canvas could remain, but simply embed a CanvasImage. Though for practical purposes, I think most developers would simply use CanvasImage with ImageView from then on. jeff On Apr 23, 2014, at 10:37 AM, Scott Palmer swpal...@gmail.com wrote: Here's a quick hack based on using snapshots to make something like a Canvas that you can reuse in many places within the Scene. By subclassing WritableImage it comes out to being pretty close to what you were talking about. As you mentioned, using snapshots isn't ideal, but I wonder how far from ideal this really is? Having to remember to make the snapshot is just one call. public class CanvasImage extends WritableImage { private final Canvas canvas; private final SnapshotParameters snapshotParams; public CanvasImage(int width, int height) { super(width, height); canvas = new Canvas(width, height); snapshotParams = new SnapshotParameters(); snapshotParams.setFill(Color.TRANSPARENT); } public GraphicsContext beginDrawing() { return canvas.getGraphicsContext2D(); } public void endDrawing() { canvas.snapshot(snapshotParams, this); } } Cheers, Scott On Tue, Apr 22, 2014 at 7:01 PM, Scott Palmer swpal...@gmail.com wrote: It would be great if you could get a Graphics context for drawing into a WritableImage instead of having to deal with snapshots. But I suppose you could build something like that based on Canvas that would update an image. Scott On Apr 22, 2014 6:52 PM, Jeff Martin j...@reportmill.com wrote: Just by using it as an ImageView.Image. jeff On Apr 22, 2014, at 5:43 PM, John C. Turnbull ozem...@ozemail.com.au wrote: How would you (easily) use it in the scenegraph if it wasn't a Node? -Original Message- From: openjfx-dev [mailto:openjfx-dev-boun...@openjdk.java.net] On Behalf Of Jeff Martin Sent: Wednesday, April 23, 2014 8:16 AM To: openjfx-dev@openjdk.java.net Mailing Subject: Should Canvas have been Image subclass instead of Node? I have a case where I need to draw to a canvas and reuse it in multiple nodes. My non-optimal work-around is to take a snapshot and use that, but it makes me wonder if Canvas should have been an Image subclass or if WritableImage should get it's own getGraphicsContext() method. jeff=
hg: openjfx/8u-dev/rt: RT-36808 [Monocle] LinuxMouseProcessor sends duplicate state updates
Changeset: 37b4ef736543 Author:Daniel Blaukopf daniel.blauk...@oracle.com Date: 2014-04-23 20:30 +0300 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/37b4ef736543 RT-36808 [Monocle] LinuxMouseProcessor sends duplicate state updates Reviewed-by: kselle ! modules/graphics/src/main/java/com/sun/glass/ui/monocle/linux/LinuxMouseProcessor.java ! tests/system/src/test/java/com/sun/glass/ui/monocle/input/SimpleMouseTest.java
Re: Should Canvas have been Image subclass instead of Node?
I think a graphics context on a writable image would make more sense. In any case, Canvas is a Node and will remain one forever. Steve On 2014-04-22 6:15 PM, Jeff Martin wrote: I have a case where I need to draw to a canvas and reuse it in multiple nodes. My non-optimal work-around is to take a snapshot and use that, but it makes me wonder if Canvas should have been an Image subclass or if WritableImage should get it's own getGraphicsContext() method. jeff
Re: Should Canvas have been Image subclass instead of Node?
I understand that Canvas has to remain a node - but I think the basic problem could be fixed in a very backward compatible way: A future release could simply add a CanvasImage class which had a GraphicsContext - then Canvas could remain, but simply embed a CanvasImage. Though for practical purposes, I think most developers would simply use CanvasImage with ImageView from then on. jeff On Apr 23, 2014, at 2:15 PM, Stephen F Northover steve.x.northo...@oracle.com wrote: I think a graphics context on a writable image would make more sense. In any case, Canvas is a Node and will remain one forever. Steve On 2014-04-22 6:15 PM, Jeff Martin wrote: I have a case where I need to draw to a canvas and reuse it in multiple nodes. My non-optimal work-around is to take a snapshot and use that, but it makes me wonder if Canvas should have been an Image subclass or if WritableImage should get it's own getGraphicsContext() method. jeff
hg: openjfx/8u-dev/rt: 3 new changesets
Changeset: 4c626e6cf0fd Author:Felipe Heidrich felipe.heidr...@oracle.com Date: 2014-04-23 09:14 -0700 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/4c626e6cf0fd RT-36636: [Accessibility] Hit test and range bounds implemented on Mac ! modules/graphics/src/main/java/com/sun/glass/ui/mac/MacAccessible.java ! modules/graphics/src/main/java/com/sun/glass/ui/mac/MacVariant.java Changeset: e8ee46b11faf Author:Felipe Heidrich felipe.heidr...@oracle.com Date: 2014-04-23 10:39 -0700 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/e8ee46b11faf [Accessibility] remove tabs from code ! modules/graphics/src/main/native-glass/mac/GlassAccessible.m Changeset: f5d934a037a8 Author:Felipe Heidrich felipe.heidr...@oracle.com Date: 2014-04-23 15:54 -0700 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/f5d934a037a8 RT-36636: [Accessibility] Mac font support added - had to implement NSAttributedString and NSDictionary ! modules/graphics/src/main/java/com/sun/glass/ui/mac/MacAccessible.java ! modules/graphics/src/main/java/com/sun/glass/ui/mac/MacVariant.java ! modules/graphics/src/main/native-glass/mac/GlassAccessible.h ! modules/graphics/src/main/native-glass/mac/GlassAccessible.m
hg: openjfx/8u-dev/rt: [EXPERIMENTAL] WebTerminal: Fix javadoc to avoid warnings.
Changeset: 5af944034d68 Author:Per Bothner per.both...@oracle.com Date: 2014-04-23 16:05 -0700 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/5af944034d68 [EXPERIMENTAL] WebTerminal: Fix javadoc to avoid warnings. ! apps/experiments/WebTerminal/src/webterminal/WebTerminal.java
hg: openjfx/8u-dev/rt: 5 new changesets
Changeset: a18b7ab2d4b5 Author:jgiles Date: 2014-04-23 14:47 +1200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/a18b7ab2d4b5 RT-36652: [ComboBox] Selected color is gray, not blue as expected ! modules/controls/src/main/resources/com/sun/javafx/scene/control/skin/modena/modena.css Changeset: 645a119142e1 Author:jgiles Date: 2014-04-24 10:44 +1200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/645a119142e1 RT-36800: [ListView] IndexOutOfBoundsException when navigating with keyboard (regression) ! modules/controls/src/main/java/com/sun/javafx/scene/control/behavior/ListViewBehavior.java ! modules/controls/src/main/java/com/sun/javafx/scene/control/behavior/TreeViewBehavior.java ! modules/controls/src/main/java/javafx/scene/control/MultipleSelectionModelBase.java ! modules/controls/src/test/java/javafx/scene/control/ListViewKeyInputTest.java ! modules/controls/src/test/java/javafx/scene/control/TableViewKeyInputTest.java ! modules/controls/src/test/java/javafx/scene/control/TreeTableViewKeyInputTest.java ! modules/controls/src/test/java/javafx/scene/control/TreeViewKeyInputTest.java Changeset: 682a4407c437 Author:jgiles Date: 2014-04-24 13:46 +1200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/682a4407c437 RT-36714: Flickering empty rows while updating a cell in a ListView ! modules/controls/src/main/java/com/sun/javafx/scene/control/skin/ListViewSkin.java ! modules/controls/src/main/java/com/sun/javafx/scene/control/skin/TableViewSkinBase.java ! modules/controls/src/main/java/com/sun/javafx/scene/control/skin/VirtualFlow.java Changeset: 5044fc222f5d Author:jgiles Date: 2014-04-24 14:47 +1200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/5044fc222f5d RT-36445: -fx-highlight-text-fill is hardcoded for .text-input:focused Contributed-by: James Denvir jim.den...@gmail.com Reviewed-by: jgiles, dgrieve, leifs ! modules/controls/src/main/resources/com/sun/javafx/scene/control/skin/modena/modena.css Changeset: b48ca56f8dec Author:jgiles Date: 2014-04-24 15:18 +1200 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/b48ca56f8dec RT-34499: [TitledPane] Fixed minimum height in TitledPane header ! apps/toys/Hello/src/main/java/hello/HelloTitledPane.java ! modules/controls/src/main/java/com/sun/javafx/scene/control/skin/TitledPaneSkin.java ! modules/controls/src/test/java/javafx/scene/control/AccordionTest.java
hg: openjfx/8u-dev/rt: Fix to RT-36485: Uninitialised memory in rt/modules/graphics/src/main/native-iio/libjpeg7
Changeset: 9305a30e8313 Author:Chien Yang chien.y...@oracle.com Date: 2014-04-23 21:59 -0700 URL: http://hg.openjdk.java.net/openjfx/8u-dev/rt/rev/9305a30e8313 Fix to RT-36485: Uninitialised memory in rt/modules/graphics/src/main/native-iio/libjpeg7 Reviewed by kcr ! modules/graphics/src/main/native-iio/libjpeg7/jchuff.c ! modules/graphics/src/main/native-iio/libjpeg7/jcmaster.c ! modules/graphics/src/main/native-iio/libjpeg7/jdhuff.c