[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-openjfx-dev
Subject: Re: RFR: 8092102: Labeled: truncated property [v8]
From: Andy Goryachev <angorya () openjdk ! org>
Date: 2024-03-28 22:02:38
Message-ID: MS0yayXImpZr-oR1SmN3XV2P4nzcrHHlusZrkilrxLM=.d556b6a9-1264-4dbb-8118-ba71ef3510c7 () github ! com
[Download RAW message or body]
On Thu, 28 Mar 2024 21:44:49 GMT, Andy Goryachev <angorya@openjdk.org> wrote:
> > Adds **Labeled.textTruncated** property which indicates when the text is visually \
> > truncated (and the ellipsis string is inserted) in order to fit the available \
> > width.
> > The new property reacts to changes in the following properties:
> > - ellipsisString
> > - font
> > - height
> > - text
> > - width
> > - wrapText
> >
> > I don't think it's worth creating a headful test (headless won't work) due to \
> > relative simplicity of the code.
> > **Alternative**
> >
> > The desired functionality can be just as easily achieved on an application level, \
> > by adding a similar property to a subclass. What is the benefit of adding this \
> > functionality to the core?
> > UPDATE 2024/03/07: turns out Labeled in a TableView (in a TreeTableView as well) \
> > lives by different rules (TableCellSkinBase:152, TreeTableCellSkin:126). The \
> > consequence of this is that the new functionality **cannot** be fully implemented \
> > with the public APIs alone.
> > **See Also**
> >
> > * [JDK-8327483](https://bugs.openjdk.org/browse/JDK-8327483) TreeView: Allow for \
> > tooltip when cell text is truncated
> > * [JDK-8205211](https://bugs.openjdk.org/browse/JDK-8205211) Ability to show \
> > Tooltip only when text is shown with ellipsis (...)
>
> Andy Goryachev has updated the pull request incrementally with one additional \
> commit since the last revision:
> add exports
unit tests added, please re-review.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1389#issuecomment-2026206381
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic