On Thu, 20 Nov 2025 17:58:53 GMT, Jurgen <[email protected]> wrote: >> The expectation is that most text cells contain `TextFlow`, with the `VFlow` >> managing the layout. Once `VFlow` determines the target width, it queries >> the `TextFlow` for its preferred height to determine the actual size. >> >> For paragraphs that contains `Regions` - either inline or as whole >> paragraphs - this logic needs to be augmented to propagate the >> `requestLayout()` flag up the hierarchy. We don't want to do this for pure >> text cells, but we must do it for embedded nodes. >> >> `addNode(Node)` handles the inline node case, >> `RichParagraph.of(Supplier<Region>)` does the "full-width" case. >> >> so to answer your question - if I understand the issue - the `embedsNode` >> flag was added to enable telling `VFlow` that it needs to reflow because >> some embedded node asked for it. Pure text cells don't need this signaling >> enabled. >> >> Did I answer your question (satisfactorily :-) ? > > I think so .... >> Pure text cells don't need this signaling enabled. > > If this is what is intended, then from how I read the code that is not what > is happening because `add(Node)` is called even for Text only cells. See: > > https://github.com/openjdk/jfx/blob/f87448ec156608527d77a4204e98e08052ffecd1/modules/jfx.incubator.richtext/src/main/java/com/sun/jfx/incubator/scene/control/richtext/VFlow.java#L827-L828
Thanks again @Jugen for pointing this out. Sorry, it took a while for me to understand what's going on. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1975#discussion_r2547416975
