On Thu, 5 Mar 2020 21:47:35 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> the api is in a related issue (should be noted in the issue above). The >> use-cases mentioned in the isssue: it's a do-once scenario with the skin >> (and its subclasses) as the only user: multiple listeners can be registered, >> typically by a skin and/or its subclasses, each can be pre/postpended (or >> replaced) in the chain for that property and all are unregistered at dispose >> time. So I think it's really bad to use it in another collaborator like the >> indicator, particularly if that is re-created often and each of the >> instances need to remove the listener. > > I think this will be fine. The `text` node is private to the > `DeterminateIndicator` inner class and not exposed, so there would never be > another listener. ahh yeah, you are right - somehow missed that text is private to the private indicator ------------- PR: https://git.openjdk.java.net/jfx/pull/71
