Skip to content

Avoid stray mouse-click after choosing date with calendar#133

Merged
humdingerb merged 1 commit intoHaikuArchives:masterfrom
humdingerb:strayclick
Jan 2, 2026
Merged

Avoid stray mouse-click after choosing date with calendar#133
humdingerb merged 1 commit intoHaikuArchives:masterfrom
humdingerb:strayclick

Conversation

@humdingerb
Copy link
Member

Delay closing of calendar window to avoid stray clicks in list of transactions.

Fixes #132

@korli
Copy link
Contributor

korli commented Dec 21, 2025

Maybe the SendMessage should be synchronous?

@humdingerb
Copy link
Member Author

Please have a look at the latest change. That's what you meant, right?
Unfortunately, this still has the transaction list behind the calendar window receiving a click from the double-click that chose the date in the calendar. Which is weird, innit? Shouldn't the double-click only be received by the calendar view, which determines the date, then the calendar window gets closed. How does the "echo" of the double-click reach the view with the transaction list at all?

@korli
Copy link
Contributor

korli commented Dec 22, 2025

Please have a look at the latest change. That's what you meant, right? Unfortunately, this still has the transaction list behind the calendar window receiving a click from the double-click that chose the date in the calendar. Which is weird, innit? Shouldn't the double-click only be received by the calendar view, which determines the date, then the calendar window gets closed. How does the "echo" of the double-click reach the view with the transaction list at all?

Yes. Like you I don't understand the behavior. Maybe the message isn't consumed by the window, and the next window in focus receives it. Looks like a bug in Haiku (every input coming before the focused window is closed should be discarded IMO).

Delay closing of calendar window to avoid stray clicks in list of
transactions.

Fixes HaikuArchives#132
@humdingerb
Copy link
Member Author

I've reported the issue as https://dev.haiku-os.org/ticket/19861
I'll merge this delaying workaround for now.

@humdingerb humdingerb merged commit bfced25 into HaikuArchives:master Jan 2, 2026
4 of 5 checks passed
@humdingerb humdingerb deleted the strayclick branch January 2, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Mouse-click confirming date in calendar chooses a transaction

2 participants