RFR: 8351047: TitledPane should handle titles that are resizable
Michael Strauß
mstrauss at openjdk.org
Sat Mar 29 07:15:07 UTC 2025
On Wed, 26 Mar 2025 21:48:23 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> That was just for deriving the `hpos` and `vpos` fields, which are no longer needed, in case `pos` was `null`. The `null` check is now handled in `LabeledSkinBase`.
>>
>> Of course we could think of some kind of derive option (how to derive LEFT + RIGHT?) or a way to remove one of these, but I think either way, it's outside the scope of this change. I added the comment for future maintainers (I could have left it out, and we'd probably not be having this discussion).
>>
>> As it is, I don't think we should fix this as part of this change as it is really completely unrelated (and might need much more discussion and perhaps even a CSR...)
>>
>> Perhaps I should remove the comment and instead we describe this in an issue?
>
> I agree: this change is out of scope for this PR.
I suggest to remove the comment and file a JBS ticket.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1742#discussion_r2019741389
More information about the openjfx-dev
mailing list