On Tue, 23 Jul 2024 14:52:02 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> modules/javafx.graphics/src/main/java/javafx/scene/AccessibleAttribute.java >> line 694: >> >>> 692: /** >>> 693: * Returns the text selection start offset for the node. >>> 694: * Selection start corresponds to the smaller offset in the >>> selection range. >> >> Just an observation, The selection is represented by IndexRange where we >> refer the start and end as **index**. But here in this file, we refer them >> as offset. > > Hmm, actually, `IndexRange.getStart()` L76 and `.getEnd()` L84 refer to > start/end _position_. I was just sharing an observation, wasn't expecting any change. In this file, documentation already refer them as offset at a few places, like BOUNDS_FOR_RANGE, LINE_START, LINE_END. So, offset seems more suitable in this file. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1512#discussion_r1689401982