Re: RFR: 8357393: RichTextArea: fails to properly save text attributes [v2]

2025-05-29 Thread Andy Goryachev
> Fixing a bug that breaks proper serialization of character attributes. > > This, unfortunately, makes a breaking change in the RichTextArea native > format [0]. > > ## References > > [0] > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/RichTextArea/Ri

Re: RFR: 8353599: TabPaneSkin: add 'menuGraphicFactory' property [v3]

2025-05-29 Thread Andy Goryachev
; tabPane.getTabs().addAll(tab1, tab2); > > TabPaneSkin skin = new TabPaneSkin(tabPane); > // set overflow menu factory with the same method as was used to > create the tabs > skin.setMenuGraphicFactory(this::createGraphic); > tabPane.setSkin(skin

Re: RFR: 8345348: CSS media feature queries [v27]

2025-05-29 Thread Andy Goryachev
On Thu, 29 May 2025 15:47:20 GMT, Michael Strauß wrote: >> Implementation of [CSS media >> queries](https://gist.github.com/mstr2/cbb93bff03e073ec0c32aac317b22de7). > > Michael Strauß has updated the pull request incrementally with one additional > commit since the last revision: > > reorder

Re: CFV: New OpenJFX Committer: Alexander Zuev

2025-05-28 Thread Andy Goryachev
Vote: YES -andy From: openjfx-dev on behalf of Kevin Rushforth Date: Wednesday, May 28, 2025 at 12:51 To: openjfx-dev , Alexander Zuev Subject: CFV: New OpenJFX Committer: Alexander Zuev I hereby nominate Alexander Zuev [1] to OpenJFX Committer. Alexander is a member of the JavaFX team at

Re: RFR: 8313424: JavaFX controls in the title bar (Preview) [v75]

2025-05-28 Thread Andy Goryachev
On Tue, 20 May 2025 19:01:26 GMT, Michael Strauß wrote: >> Implementation of >> [`StageStyle.EXTENDED`](https://gist.github.com/mstr2/0befc541ee7297b6db2865cc5e4dbd09). > > Michael Strauß has updated the pull request with a new target base due to a > merge or a rebase. The pull request now cont

Re: RFR: 8207333: [Linux, macOS] Column sorting is triggered always after context menu request on table header [v5]

2025-05-28 Thread Andy Goryachev
On Mon, 28 Apr 2025 18:27:37 GMT, Jose Pereda wrote: >> Note: The JBS issue >> [JDK-8207333](https://bugs.openjdk.org/browse/JDK-8207333) refers to Linux, >> but it happens on macOS too. >> >> When a TableColumn has a ContextMenu, if the user right clicks on the >> tableColumnHeader, the Con

Re: CodeArea: request to raise priority of JDK-8357405 (text length metrics for ContentChange)

2025-05-27 Thread Andy Goryachev
Dear Pavel: This is not a trivial request. Currently, there is not easy way to control (override) the behavior of the CodeModel with respect to line endings. Line endings are not stored in the model, but they are emitted when saving/copying. Perhaps we ought to add a dedicated property to the

Re: RFR: 8345348: CSS media feature queries [v25]

2025-05-27 Thread Andy Goryachev
On Tue, 27 May 2025 17:18:47 GMT, Michael Strauß wrote: >> modules/javafx.graphics/src/main/java/javafx/scene/Scene.java line 6571: >> >>> 6569: * Media Feature >>> 6570: * >>> 6571: * >>> Name-fx-prefers-persistent-scrollbars >> >> why the change to

Re: RFR: 8345348: CSS media feature queries [v25]

2025-05-27 Thread Andy Goryachev
On Sun, 25 May 2025 00:19:36 GMT, Michael Strauß wrote: >> Implementation of [CSS media >> queries](https://gist.github.com/mstr2/cbb93bff03e073ec0c32aac317b22de7). > > Michael Strauß has updated the pull request with a new target base due to a > merge or a rebase. The pull request now contains

Re: RFR: 8341281: Root TreeItem with null value breaks TreeTableView [v4]

2025-05-27 Thread Andy Goryachev
On Tue, 27 May 2025 16:09:52 GMT, Ziad El Midaoui wrote: >> When the Root TreeItem is set to null, need to relayout to show the children >> items > > Ziad El Midaoui has updated the pull request incrementally with one > additional commit since the last revision: > > Minor change : new line

Re: RFR: 8341281: Root TreeItem with null value breaks TreeTableView [v3]

2025-05-27 Thread Andy Goryachev
On Tue, 27 May 2025 14:03:15 GMT, Ziad El Midaoui wrote: >> When the Root TreeItem is set to null, need to relayout to show the children >> items > > Ziad El Midaoui has updated the pull request incrementally with two > additional commits since the last revision: > > - Merge remote-tracking

Re: RFR: 8356983: Create implementation of NSAccessibilityImage protocol

2025-05-27 Thread Andy Goryachev
er: https://github.com/andy-goryachev-oracle/MonkeyTest VoiceOver announces the accessible text set on ImageView (the image view was set focus traversable). Also the following is being logged when Logging -> Accessibility is turned on: {time:1748360254874, log:"accessibility", "

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v31]

2025-05-27 Thread Andy Goryachev
> Please refer to > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Text/LayoutInfo.md > > The RichTextArea control > ([JDK-8301121](https://bugs.openjdk.org/browse/JDK-8301121)), or any custom > control that needs non-trivial navigation within complex or wr

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v30]

2025-05-27 Thread Andy Goryachev
On Thu, 22 May 2025 21:47:22 GMT, Michael Strauß wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 59 commits: >> >> - Merge remote-tracking branch 'origin/master' int

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v27]

2025-05-22 Thread Andy Goryachev
On Thu, 22 May 2025 17:59:40 GMT, Andy Goryachev wrote: >> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 1123: >> >>> 1121: * @since 25 >>> 1122: */ >>> 1123: public final PathElement[] getStrikeThroughShape(int start

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v30]

2025-05-22 Thread Andy Goryachev
> Please refer to > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Text/LayoutInfo.md > > The RichTextArea control > ([JDK-8301121](https://bugs.openjdk.org/browse/JDK-8301121)), or any custom > control that needs non-trivial navigation within complex or wr

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v29]

2025-05-22 Thread Andy Goryachev
> Please refer to > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Text/LayoutInfo.md > > The RichTextArea control > ([JDK-8301121](https://bugs.openjdk.org/browse/JDK-8301121)), or any custom > control that needs non-trivial navigation within complex or wr

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v27]

2025-05-22 Thread Andy Goryachev
On Thu, 22 May 2025 00:05:29 GMT, Kevin Rushforth wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 55 commits: >> >> - review comments >> - Merge remote-tracki

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v28]

2025-05-22 Thread Andy Goryachev
> Please refer to > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Text/LayoutInfo.md > > The RichTextArea control > ([JDK-8301121](https://bugs.openjdk.org/browse/JDK-8301121)), or any custom > control that needs non-trivial navigation within complex or wr

Re: [External] : richtextarea / documentation / feedback

2025-05-22 Thread Andy Goryachev
ted https://bugs.openjdk.org/browse/JDK-8357582 to rectify that. Hope this helps, and thank you for the feedback - keep it coming! -andy From: Timo Date: Thursday, May 22, 2025 at 04:18 To: Andy Goryachev Subject: [External] : richtextarea / documentation / feedback Hi Andy Thanks for the richtextare

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v18]

2025-05-22 Thread Andy Goryachev
On Thu, 22 May 2025 15:05:02 GMT, Michael Strauß wrote: >> I would have agreed with the proposed change, if it were a public API. >> >> This is an internal method, so it was made as low level as possible. The >> return value is documented, and the consumer is another part of internal >> code.

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v27]

2025-05-22 Thread Andy Goryachev
On Thu, 22 May 2025 00:05:29 GMT, Kevin Rushforth wrote: >> Andy Goryachev has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 55 commits: >> >> - review comments >> - Merge remote-tracki

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v18]

2025-05-22 Thread Andy Goryachev
On Wed, 21 May 2025 23:49:29 GMT, Michael Strauß wrote: >> This method does not throw any exceptions, so none are specified. > > Instead of returning `float[]` here, this would be a good candidate to return > a discriminated union instead, something like: > > sealed interface CaretGeometry { >

Re: RFR: 8341670: [Text, TextFlow] Public API for Text Layout Info [v27]

2025-05-21 Thread Andy Goryachev
> Please refer to > > https://github.com/andy-goryachev-oracle/Test/blob/main/doc/Text/LayoutInfo.md > > The RichTextArea control > ([JDK-8301121](https://bugs.openjdk.org/browse/JDK-8301121)), or any custom > control that needs non-trivial navigation within complex or wr

RFR: 8357393: RichTextArea: fails to properly save text attributes

2025-05-21 Thread Andy Goryachev
Fixing a bug that breaks proper serialization of character attributes. This, unfortunately, makes a breaking change in the RichTextArea native format [0]. ## References [0] https://github.com/andy-goryachev-oracle/Test/blob/main/doc/RichTextArea/RichTextArea_DataFormat_V2.md

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v7]

2025-05-21 Thread Andy Goryachev
On Wed, 21 May 2025 10:22:10 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's widt

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v6]

2025-05-20 Thread Andy Goryachev
On Tue, 20 May 2025 15:53:53 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's widt

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]

2025-05-20 Thread Andy Goryachev
On Tue, 20 May 2025 09:35:45 GMT, Ambarish Rapte wrote: >>> Would it be possible to avoid re-layout of the cells that are after the >>> Cell with a disclosureNode ? >> >> This is actually what is happening, when we scroll down and find a TreeCell >> that has a disclosure node this is when the

Integrated: 8355415: RichTextArea: NPE in VFlow::scrollCaretToVisible

2025-05-20 Thread Andy Goryachev
On Wed, 23 Apr 2025 22:05:33 GMT, Andy Goryachev wrote: > Fixed an NPE that should only happen in a headless test environment. > > This is a follow-up for https://github.com/openjdk/jfx/pull/1677 This pull request has now been integrated. Changeset: 9ca61edb Author:Andy Gorya

Re: RFR: 8355415: RichTextArea: NPE in VFlow::scrollCaretToVisible [v2]

2025-05-20 Thread Andy Goryachev
On Tue, 20 May 2025 17:00:33 GMT, Ambarish Rapte wrote: >> Andy Goryachev 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 contai

Re: Let's discuss JDK-8338952

2025-05-20 Thread Andy Goryachev
etSortPolicy(sortPolicy); But don't you think the VALUE parameters are redundant since they can be obtained directly from ITEM? Best regards, Pavel On 5/19/25 22:28, Andy Goryachev wrote: One possible solution would be to deprecate the comparator property (not for removal, keep it for ba

RFR: 8314482: TextFlow: TabStopPolicy

2025-05-19 Thread Andy Goryachev
# Tab Stop Policy Andy Goryachev ## Summary Introduce a `tabStopPolicy` property in the `TextFlow` class which, when set, overrides the existing `tabSize` value and provides consistent way of setting tab stops at the paragraph level, regardless of the individual text segments font [0

Re: Let's discuss JDK-8338952

2025-05-19 Thread Andy Goryachev
One possible solution would be to deprecate the comparator property (not for removal, keep it for backward compatibility), and add a new property 'sortPolicy' which will use an interface that can be evolved, as opposed to a naive callback. public interface SortPolicy { public int compare(IT

Re: CodeArea: how to convert offset to TextPos?

2025-05-19 Thread Andy Goryachev
vel On 5/7/25 20:04, Andy Goryachev wrote: There is no dedicated method to do that. The application can iterate over paragraphs to compute the text position, just keep in mind that the model can be quite large and it a) will take a lot of iterations and b) the result may exceed 2^31 (i.e. need t

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-19 Thread Andy Goryachev
On Mon, 19 May 2025 18:05:39 GMT, Michael Strauß wrote: >> 100 still looks excessive to me - it's unlikely that we'll have two >> different scenarios when this happens, so most of the time the log will have >> 100 identical traces. >> >> I mean, one is probably enough, we could have 2 or 4 jus

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v3]

2025-05-19 Thread Andy Goryachev
On Sat, 17 May 2025 12:24:46 GMT, Michael Strauß wrote: >> When an exception is thrown from `AnimationTimer::handle`, the JavaFX >> application freezes. The reason is that the user exception will bubble up >> into framework code, preventing the normal operation of JavaFX. >> >> The following p

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-19 Thread Andy Goryachev
On Sat, 17 May 2025 12:21:17 GMT, Michael Strauß wrote: >> On the other hand, AnimationTimer is a public API that allows for arbitrary >> code execution, which always has the potential to fail for any number of >> reasons (think of calling an unstable API from the timer callback). We can’t >>

Re: RFR: 8238435: [macOs] Remove use of CGEventTap

2025-05-19 Thread Andy Goryachev
On Sun, 18 May 2025 19:02:14 GMT, Martin Fox wrote: > The GlassTouches code for monitoring touch events was disabled years ago > because it uses a global event tap. As Apple tightened security this started > generating alerts. With the latest version of Xcode (16.3) this unused code > is trigg

Re: RFR: 8356652: Input field ignores custom input source characters [v2]

2025-05-19 Thread Andy Goryachev
On Fri, 16 May 2025 22:25:20 GMT, Martin Fox wrote: >> Under the hood the Keyman input method appears as a US English keyboard >> layout. The characters attached to an NSEvent are always US English Roman >> even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends >> the corre

Re: RFR: 8356652: Input field ignores custom input source characters [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 22:25:20 GMT, Martin Fox wrote: >> Under the hood the Keyman input method appears as a US English keyboard >> layout. The characters attached to an NSEvent are always US English Roman >> even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends >> the corre

Re: RFR: 8356652: Input field ignores custom input source characters [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 22:25:20 GMT, Martin Fox wrote: >> Under the hood the Keyman input method appears as a US English keyboard >> layout. The characters attached to an NSEvent are always US English Roman >> even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends >> the corre

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 19:03:14 GMT, Michael Strauß wrote: >> We're throwing away all exceptions after a certain point (either after the >> first, or some number of exceptions after that). A potential use case would >> be debugging a large application that for some reason throws lots of >> differ

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 18:12:35 GMT, Michael Strauß wrote: >> When an exception is thrown from `AnimationTimer::handle`, the JavaFX >> application freezes. The reason is that the user exception will bubble up >> into framework code, preventing the normal operation of JavaFX. >> >> The following p

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 18:12:35 GMT, Michael Strauß wrote: >> When an exception is thrown from `AnimationTimer::handle`, the JavaFX >> application freezes. The reason is that the user exception will bubble up >> into framework code, preventing the normal operation of JavaFX. >> >> The following p

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 18:12:35 GMT, Michael Strauß wrote: >> When an exception is thrown from `AnimationTimer::handle`, the JavaFX >> application freezes. The reason is that the user exception will bubble up >> into framework code, preventing the normal operation of JavaFX. >> >> The following p

Re: RFR: 8357157: Exception thrown from AnimationTimer freezes application

2025-05-16 Thread Andy Goryachev
On Fri, 16 May 2025 14:38:20 GMT, Michael Strauß wrote: > When an exception is thrown from `AnimationTimer::handle`, the JavaFX > application freezes. The reason is that the user exception will bubble up > into framework code, preventing the normal operation of JavaFX. > > The following progra

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-16 Thread Andy Goryachev
On Wed, 14 May 2025 15:31:46 GMT, Martin Fox wrote: > Under the hood the Keyman input method appears as a US English keyboard > layout. The characters attached to an NSEvent are always US English Roman > even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends > the correct H

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 22:57:08 GMT, Steven Loomis wrote: >>> Do you think it's a bug in keyman or our code? I head the beep (correctly) >>> but then n appears, despite the keyman onscreen keyboard showing an empty >>> key. If it is our code, should there be some additional logic? >> >> Keyman is

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 19:04:26 GMT, Martin Fox wrote: > The N key in the Himyarit Musnad layout shouldn't generate a character (just > a beep) Do you think it's a bug in keyman or our code? I head the beep (correctly) but then n appears, despite the keyman onscreen keyboard showing an empty key

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Wed, 14 May 2025 15:31:46 GMT, Martin Fox wrote: > Under the hood the Keyman input method appears as a US English keyboard > layout. The characters attached to an NSEvent are always US English Roman > even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends > the correct H

Re: RFR: 8357067: Platform preference change can emit multiple notifications [v2]

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 18:56:14 GMT, Michael Strauß wrote: >> Some platform preference changes can trigger the emission of multiple >> notifications. For example, when switching from a high-contrast theme on >> Windows to the regular theme, the following notifications are emitted (log >> can be v

Re: RFR: 8357067: Platform preference change can emit multiple notifications

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 18:28:59 GMT, Michael Strauß wrote: >> modules/javafx.graphics/src/main/native-glass/win/PlatformSupport.h line 75: >> >>> 73: * Suggested aggregation delay for changes that come in over a >>> period of time. >>> 74: */ >>> 75: static constexpr int SUGGESTED_DE

Re: RFR: 8357067: Platform preference change can emit multiple notifications

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 17:50:50 GMT, Michael Strauß wrote: > Some platform preference changes can trigger the emission of multiple > notifications. For example, when switching from a high-contrast theme on > Windows to the regular theme, the following notifications are emitted (log > can be viewe

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Wed, 14 May 2025 15:31:46 GMT, Martin Fox wrote: > Under the hood the Keyman input method appears as a US English keyboard > layout. The characters attached to an NSEvent are always US English Roman > even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends > the correct H

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Thu, 15 May 2025 17:26:39 GMT, Alexander Zuev wrote: >> Noticed a bit of a problem. >> >> To reproduce, install Himyarit Musnad keyboard >> https://keyman.com/keyboards/himyarit_musnad >> >> typing 'h' causes this character to appear: 𐩱 >> typing 'n' followed by 'h' generates garbage input

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Wed, 14 May 2025 15:31:46 GMT, Martin Fox wrote: > Under the hood the Keyman input method appears as a US English keyboard > layout. The characters attached to an NSEvent are always US English Roman > even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends > the correct H

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-15 Thread Andy Goryachev
On Wed, 14 May 2025 15:31:46 GMT, Martin Fox wrote: > Under the hood the Keyman input method appears as a US English keyboard > layout. The characters attached to an NSEvent are always US English Roman > even if the selected Keyman layout is, say, Hebrew or Dvorak. Keyman sends > the correct H

Re: RFR: 8356652: Input field ignores custom input source characters

2025-05-14 Thread Andy Goryachev
the correct Hebrew or Dvorak character to insertText:replacementRange: > instead. > > This PR special-cases the Keyman layout, detecting it using the same method > that AWT does. When Keyman is active Glass records the insertText: character > and uses that when sending out K

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]

2025-05-14 Thread Andy Goryachev
On Wed, 14 May 2025 16:40:36 GMT, Ambarish Rapte wrote: >>> In that case the TreeCells are layout while the scroll is in progress. >> >> does this create any visual artifacts? > >> does this create any visual artifacts? > > No artifacts, just that one can notice the re-layout and the TreeCell s

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]

2025-05-14 Thread Andy Goryachev
On Wed, 14 May 2025 14:43:05 GMT, Ziad El Midaoui wrote: >> Also noticed a scenario: When a TreeCell with disclosureNode is currently >> not shown on screen but becomes visible on scrolling. In that case the >> TreeCells are layout while the scroll is in progress. I see only one way to >> avo

Re: RFR: 8355615: ConcurrentModificationException creating MenuBar on background thread [v2]

2025-05-14 Thread Andy Goryachev
On Fri, 2 May 2025 15:44:03 GMT, Andy Goryachev wrote: >> Moving MenuBarSkin initialization code to install() which always happens in >> the FX Application Thread. >> >> Also, ensure that the reverse process also happens in the FX application >> thread. > >

Integrated: 8355615: ConcurrentModificationException creating MenuBar on background thread

2025-05-14 Thread Andy Goryachev
On Fri, 25 Apr 2025 22:39:12 GMT, Andy Goryachev wrote: > Moving MenuBarSkin initialization code to install() which always happens in > the FX Application Thread. > > Also, ensure that the reverse process also happens in the FX application > thread. This pull request has now

Re: RFR: 8089080: [TextArea] Caret disappear after pressing backspace to clear the content

2025-05-13 Thread Andy Goryachev
On Mon, 21 Apr 2025 15:42:20 GMT, Ziad El Midaoui wrote: > Fixed Caret disappear after unfocus then focus on the TextArea, the issue was > happening because the caret position was set to -1 when we focus on the > TextArea with empty text and caused the caret to disappear from at the skin > le

Re: RFR: 8089080: [TextArea] Caret disappear after pressing backspace to clear the content

2025-05-13 Thread Andy Goryachev
On Mon, 21 Apr 2025 15:42:20 GMT, Ziad El Midaoui wrote: > Fixed Caret disappear after unfocus then focus on the TextArea, the issue was > happening because the caret position was set to -1 when we focus on the > TextArea with empty text and caused the caret to disappear from at the skin > le

Re: JavaFX Presentation at JavaOne

2025-05-13 Thread Andy Goryachev
And a thread on reddit: https://www.reddit.com/r/java/s/IcJhCpNt7y (Warning: S/N ratio) -andy From: openjfx-dev on behalf of Nir Lisker Date: Saturday, May 10, 2025 at 22:33 To: openjfx-dev Subject: JavaFX Presentation at JavaOne Just saw Kevin's JavaFX talk released on YT. Thanks Kevin! htt

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-12 Thread Andy Goryachev
sue with ID: 9078500. Best regards, Pavel On 5/12/25 19:58, Andy Goryachev wrote: JDK-8355774 adds methods to style highlights added via addHighlight() with CSS. What (I think) you want is an ability to style a single text segment with (background | underlines | etc) attributes, which is still

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-12 Thread Andy Goryachev
does not consider the possibility of styling underline/wavy underline via a CSS file (without calling special methods like addHighlight, addWavyUnderline? If so, should I open another issue? Best regards, Pavel On 5/12/25 19:41, Andy Goryachev wrote: Yes, I know. But we must support both

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-12 Thread Andy Goryachev
I’ve said many times, I’m only considering styling through a CSS file. Best regards, Pavel On 5/12/25 19:28, Andy Goryachev wrote: Pavel: Underlines and strike-through attributes are available via StyleAttributeMap.STRIKE_THROUGH and StyleAttributeMap.UNDERLINE (using text color). On the

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-12 Thread Andy Goryachev
that only 60% of the core properties are supported. That's just my feedback — the decision is, of course, yours :) Best regards, Pavel On 5/9/25 00:09, Andy Goryachev wrote: The only property that makes sense is the background color. The wavy and other kinds of underlines, while they make so

Re: RFR: 8355615: ConcurrentModificationException creating MenuBar on background thread [v2]

2025-05-12 Thread Andy Goryachev
On Fri, 2 May 2025 15:44:03 GMT, Andy Goryachev wrote: >> Moving MenuBarSkin initialization code to install() which always happens in >> the FX Application Thread. >> >> Also, ensure that the reverse process also happens in the FX application >> thread. > >

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v5]

2025-05-12 Thread Andy Goryachev
On Fri, 9 May 2025 10:51:20 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-08 Thread Andy Goryachev
talic, size, name) (+) - underline (e.g. for links) (?) - wavy underline (e.g. for spelling) (-) I think these properties cover 99% of all the necessary properties for CodeArea. I used + and - to show what has already been implemented. Best regards, Pavel On 5/8/25 22:21, Andy Goryachev wrote: Oh, I fi

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v4]

2025-05-08 Thread Andy Goryachev
On Thu, 8 May 2025 20:16:12 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-08 Thread Andy Goryachev
e used, an alternative property can be introduced, such as -fx-highlight-color. However, everything else in the example should work as described. Best regards, Pavel On 5/8/25 21:25, Andy Goryachev wrote: Pavel: I think your comments in the ticket are still invisible to me, so I can't comment. Ab

Re: RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-08 Thread Andy Goryachev
ished CSS norms. This is my personal opinion, which, of course, can be wrong. Best regards, Pavel On 5/8/25 20:46, Andy Goryachev wrote: > Adding missing APIs related to styling the highlights with CSS: > > ![Screenshot 2025-05-07 at 15 08 > 53](https://github.com/user-attachments/ass

Re: RFR: 8355415: RichTextArea: NPE in VFlow::scrollCaretToVisible [v2]

2025-05-08 Thread Andy Goryachev
> Fixed an NPE that should only happen in a headless test environment. > > This is a follow-up for https://github.com/openjdk/jfx/pull/1677 Andy Goryachev has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelate

RFR: 8355774: RichTextArea: provide mechanism for CSS styling of highlights

2025-05-08 Thread Andy Goryachev
Adding missing APIs related to styling the highlights with CSS: ![Screenshot 2025-05-07 at 15 08 53](https://github.com/user-attachments/assets/e37062b4-9804-40a7-872d-830fe0f584c1) Adds methods to the `RichParagraph.Builder`: /** * Adds a wavy underline (typically used as a

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v2]

2025-05-08 Thread Andy Goryachev
On Wed, 7 May 2025 15:56:57 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v3]

2025-05-08 Thread Andy Goryachev
On Thu, 8 May 2025 14:59:40 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning [v2]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:56:57 GMT, Ziad El Midaoui wrote: >> The issue occurred because items preceding an item with children (items with >> a disclosure node) had different widths, which led to misalignment. This can >> be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8350316: Create implementation of NSAccessibilityProgressIndicator protocol [v2]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:42:03 GMT, Alexander Zuev wrote: >> Initial implementation. In order to implement progress indicator the group >> accessibility protocol has to be implemented too so this is also a fix for >> 8351773. There are commented out sections that can be used to verify the >> func

Re: CodeArea: -fx-background-color doesn't work.

2025-05-07 Thread Andy Goryachev
; to a text from from 5 to 20. We do not know IF THESE STYLES SET BACKGROUND COLOR. Or should we ask an user to name a class this way - "foo-bg"? Best regards, Pavel On 5/7/25 21:16, Andy Goryachev wrote: Thank you for the feedback! I finally got the ticket - https://

Re: CodeArea: -fx-background-color doesn't work.

2025-05-07 Thread Andy Goryachev
behalf of PavelTurk Date: Wednesday, May 7, 2025 at 10:30 To: openjfx-dev@openjdk.org Subject: Re: CodeArea: -fx-background-color doesn't work. Andy, thank you very much for your help! Best regards, Pavel On 5/7/25 20:29, Andy Goryachev wrote: This bug is still percolating through the system

Re: CodeArea: -fx-background-color doesn't work.

2025-05-07 Thread Andy Goryachev
opened an issue today, so, when it is approved there won't be any questions. Best regards, Pavel On 5/6/25 00:04, Andy Goryachev wrote: Will not happen. CSS compatibility with a third party library is not a requirement in this case. -andy From: openjfx-dev <mailto:openjfx-de

Re: RFR: 8350316: Create implementation of NSAccessibilityProgressIndicator protocol [v2]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:42:03 GMT, Alexander Zuev wrote: >> Initial implementation. In order to implement progress indicator the group >> accessibility protocol has to be implemented too so this is also a fix for >> 8351773. There are commented out sections that can be used to verify the >> func

Re: CodeArea: how to convert offset to TextPos?

2025-05-07 Thread Andy Goryachev
There is no dedicated method to do that. The application can iterate over paragraphs to compute the text position, just keep in mind that the model can be quite large and it a) will take a lot of iterations and b) the result may exceed 2^31 (i.e. need to be 'long'). The same is true for the re

Re: CodeArea: how to get the indexes of the first visible paragraph and the last one?

2025-05-07 Thread Andy Goryachev
RichTextArea.getTextPosition() should be able to give the answer. -andy From: openjfx-dev on behalf of PavelTurk Date: Wednesday, May 7, 2025 at 08:52 To: openjfx-dev@openjdk.org Subject: CodeArea: how to get the indexes of the first visible paragraph and the last one? Could anyone say how t

Re: RFR: 8350316: Create implementation of NSAccessibilityProgressIndicator protocol [v2]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:42:03 GMT, Alexander Zuev wrote: >> Initial implementation. In order to implement progress indicator the group >> accessibility protocol has to be implemented too so this is also a fix for >> 8351773. There are commented out sections that can be used to verify the >> func

Re: RFR: 8340344: The first item in TreeView is not aligned in the beginning

2025-05-07 Thread Andy Goryachev
On Wed, 19 Feb 2025 16:45:54 GMT, Ziad El Midaoui wrote: > The issue occurred because items preceding an item with children (items with > a disclosure node) had different widths, which led to misalignment. This can > be fixed by requesting a cell relayout whenever the disclosure node's width

Re: RFR: 8355012: JavaFX modena.css -fx-highlight-text-fill bug [v4]

2025-05-07 Thread Andy Goryachev
On Mon, 5 May 2025 21:28:08 GMT, Ziad El Midaoui wrote: >> The issue was happening because `-fx-highlight-text-fill` was set to white , >> so when the background color is bright (>60% brightness ) the text is not >> visible , the solution is to change the value to be dynamically set >> dependi

Re: RFR: 8345348: CSS media feature queries [v23]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:09:18 GMT, Kevin Rushforth wrote: >> yes it can: >> >> --add-opens javafx.graphics/com.sun.javafx.application.preferences=andy_test >> >> >> >> var platformPreferences = Platform.getPreferences(); >> var colorSchemeProperty = platformPreferences.colorSche

Re: RFR: 8345348: CSS media feature queries [v23]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 15:16:15 GMT, Kevin Rushforth wrote: >> okay > > Further, if you call `setAccessibile` in connection with `--add-opens`, which > your above code snippet does, you can even call _private_ methods. > > So at the risk of repeating myself, there is no concern that making this >

Re: RFR: 8345348: CSS media feature queries [v23]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 14:51:11 GMT, Kevin Rushforth wrote: >> No, an application would not be able to do that. A getter whose >> implementation happens to return a subtype in a non-exported package does >> not allow the caller of that getter to access any methods in the >> non-exported class (eve

Re: RFR: 8345348: CSS media feature queries [v23]

2025-05-07 Thread Andy Goryachev
On Wed, 7 May 2025 09:28:53 GMT, Michael Strauß wrote: >> modules/javafx.graphics/src/main/java/com/sun/javafx/application/preferences/PreferenceProperties.java >> line 258: >> >>> 256: } >>> 257: >>> 258: // This method must only be called on the FX application >>> thread. >>

Re: RFR: 8345348: CSS media feature queries [v24]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 20:23:01 GMT, Michael Strauß wrote: >> Implementation of [CSS media >> queries](https://gist.github.com/mstr2/cbb93bff03e073ec0c32aac317b22de7). > > Michael Strauß has updated the pull request incrementally with three > additional commits since the last revision: > > - cssr

Re: RFR: 8345348: CSS media feature queries [v21]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 20:33:43 GMT, Michael Strauß wrote: >> how about: >> >> - the implementation must not fire any events before the Scene is shown >> - the preferences getters should disallow calls from a background thread > > 1. That's how it is implemented. No change events before the scene i

Re: RFR: 8345348: CSS media feature queries [v20]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 20:29:29 GMT, Michael Strauß wrote: >> Thank you! >> >> I might suggest to drop `` since these bullets are somewhat of a >> distraction. > > Without it, it just looks like a wall of text to me. ok - PR Review Comment: https://git.openjdk.org/jfx/pull/1655#discu

Re: RFR: 8345348: CSS media feature queries [v21]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 20:23:10 GMT, Michael Strauß wrote: >> That's why I am against this background thread nonsense. Create - yes (even >> though I am against this). Access and modify properties, especially those >> that interact with other parts of the FX - no. >> >> I think we should enforce

Re: RFR: 8345348: CSS media feature queries [v20]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 20:18:05 GMT, Michael Strauß wrote: >> I insist: the doc should include an example with an operator, and possibly >> how to achieve A and not B case mentioned earlier. >> >> Assume the reader has no clue how media queries work and does not want to >> wade through the massive

Re: RFR: 8345348: CSS media feature queries [v21]

2025-05-06 Thread Andy Goryachev
On Tue, 6 May 2025 19:20:04 GMT, Michael Strauß wrote: >> I see. The I suggest we enforce single-threaded nature of it: everything >> must happen in the FX Application Thread. I don't think it's a good idea to >> start synchronizing things in FX now all of a sudden. >> >> Any code that wants

  1   2   3   4   5   6   7   8   9   10   >