ui: prevent checkbox click from propagating in tools submenu

[MaxKruse]

Overview

Fixes #22796

Additional information

I have absolutely no idea why preventing propagation onclick for the Checkbox fixes the issue. Frontend logic is beyond me. But a fix is a fix

Requirements

• I have read and agree with the contributing guidelines
• AI usage disclosure: YES. Had Qwen3.5 122b try to figure out whats going on exactly, then checked myself to see if there was another reason for the bug - as far as i can tell, there wasnt.

[allozaur]

I have absolutely no idea why preventing propagation onclick for the Checkbox fixes the issue. Frontend logic is beyond me. But a fix is a fix

Hmm, will see into this and get back to you.

[allozaur]

okay, so as a short term solution let's merge your fix, but please add a todo comment that this needs to be revisited and investigated in the future and make sure that the comment includes todo and hack words

[MaxKruse]

@allozaur Should be good to go, apologies for no deeper analysis on my part as to why this seems broken 😔

[allozaur]

@allozaur Should be good to go, apologies for no deeper analysis on my part as to why this seems broken 😔

np, @ServeurpersoCom i will pass this one to you to handle as u have Windows machine at hand

[ServeurpersoCom]

I think we need to refactor the logic of this tool selector (the design is already clean, and I'm doing the logic first) plus we don't need to bother with a migration since it's quick to configure. Also to fix #22796

[allozaur]

I think we need to refactor the logic of this tool selector (the design is already clean, and I'm doing the logic first) plus we don't need to bother with a migration since it's quick to configure. Also to fix #22796

oki, so i guess we can close this PR once your supersedes it? @ServeurpersoCom

[ServeurpersoCom]

I think we need to refactor the logic of this tool selector (the design is already clean, and I'm doing the logic first) plus we don't need to bother with a migration since it's quick to configure. Also to fix #22796

oki, so i guess we can close this PR once your supersedes it? @ServeurpersoCom

Yes, it's better to tackle the root cause.