It's safe to send MOUSE_PRESSED to a disabled button, since there is a guard in place that will not fire the action if the button is disabled.

sending these three in sequence from another handler

Actually, a single MOUSE_MOVE should start the timer and then show the popup.

@edvin exit is a little tricky because that will fire as the popup is shown

@User uploaded a file: Not Firing b2's Toolip

I verified that B2's tooltip works in the hover state but didn't see anything 10 seconds after this sequence was submitted. All the printlns showed up

just about to upgrade to 1.1 - any gotchas or pretty straightforward?

@edvin Are you looking at TooltipBehavior?

@gtnarg I have a few, but I'd actually like to see if you run into any problems at all. Just look at the quick version notes on the TornadoFX 1.7.0 release first!

@edvin - okay - will do...

@carlw Yes

If you don't find a better algorithm, do you want to review a tornadofx version using the live format? I'm thinking about an extension function allowdisabkedtooltip

@gtnarg I had no real issues, just compile everything (mvn clean 😆 )

sometimes code I thought was too overengineered, suddenly helps 🙂

@User uploaded a file: Untitled

basically re using the saveAs and the openRepository methods to bootstrap the new file

@edvin - shouldn’t the version be

1.7.1-SNAPSHOT

from source control? It is still

1.7.0

@gtnarg Sorry, will change ASAP. How did it go with you? Was everything OK after update to 1.7.0?

@carlw Just back now, will have a look at the tooltip thing 🙂

@edvin - np - so far so good - still updating...

@gtnarg I don't think we neeed to relay MOUSE_PRESSED. I have a suggestion now. Bear in mind that this doesn't listen to changes in the parentProperty. It probably shout, to disconnect and reconnect to the new parent, but for most use cases it's fine like it is.

@User uploaded a file: Untitled

@edvin - I think you mean @carlw

@edvin - upgrade went well - no problems 🙂

@gtnarg That's very nice to hear 🙂 I ran into so many issues in the beginning, and this horrible bug of course. Good to know that we won't create trouble for our users with this upgrade.

@carlw I don't think we neeed to relay MOUSE_PRESSED. I have a suggestion now. Bear in mind that this doesn't listen to changes in the parentProperty. It probably shout, to disconnect and reconnect to the new parent, but for most use cases it's fine like it is.

@carlw I posted the code above 🙂

ok...testing

I updated the sample three times, make sure you have the most current 🙂 Sorry 🙂

Tested it with this code: