On Wed, 20 May 2026 15:21:17 GMT, Michael Strauß <[email protected]> wrote:
>> Martin Fox has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Better comments for dispatching utility > > While this PR seems to fix an actual bug, it also runs the risk of preempting > the discussion on focus delegation / multi-level focus that we've had several > times, but never reached a conclusion. I wouldn't want this proposed solution > (directly dispatching the event to its final target) to be the last word on > the topic of focus delegation, especially when it comes to a potential > API-level solution. > @mstr2 : Unless you can bring up a scenario where the proposed solution > breaks things, we should probably keep these two things separate. > > Once we reach the agreement, it is very likely that changes will be needed > everywhere anyway. I don't want to get stuck in endless discussions that lead > nowhere, I would rather see the platform progressing. > > Do you have a possible failure mode? This depends on the definition of "failure mode". With this solution, an event listener that is installed on a node in the skin subgraph that eventually contains the `TextField` will not see the event (because it does not travel through the subgraph, it is dispatched directly to the final target). This is different from how events in the scene graph usually work. It's a small inconsistency, and it may be completely acceptable. What I'm saying is that I don't want a narrow-scoped solution as the one presented here to sneak into FX and then just by being there become the de-facto solution for focus-delegation-like problems even when we never conclusively decided how we want to solve the bigger picture. ------------- PR Comment: https://git.openjdk.org/jfx/pull/2166#issuecomment-4500067975
