Wrap toolbar items with MacosToolbarPassthrough

[cbenhagen]

Wrap toolbar items with MacosToolbarPassthrough to prevent window move or resize when interacting with toolbar items.

Pre-launch Checklist

• I have incremented the package version as appropriate and updated CHANGELOG.md with my changes
• I have added/updated relevant documentation
• I have run "optimize/organize imports" on all changed files
• I have addressed all analyzer warnings as best I could (only the ones affecting my changes)

[cbenhagen]

@Adrian-Samoticha / @GroovinChip failing checks seem unrelated to this PR.

[Adrian-Samoticha]

@Adrian-Samoticha / @GroovinChip failing checks seem unrelated to this PR.

Yeah, I believe we’ve been failing them for a while now. Thanks for the PR; we’ll review it soon!

[GroovinChip]

This looks good to me, but there's some housekeeping that needs to happen on main before it can be merged. I'll try to get to in on Monday.

[GroovinChip]

@cbenhagen In the meantime, could you please post a screenshot or gif of the change in action? Thanks.

[cbenhagen]

Hard to show in a gif. Without this change the window will maximize if you double click a toolbar item.

[GroovinChip]

@cbenhagen housekeeping has been completed. Please merge the latest changes from dev into your branch and we can move this forward.

[cbenhagen]

@GroovinChip done ;)

[GroovinChip]

Looks great, but could you please write a test that validates the change so that we can minimize potential regression? Thanks!

[cbenhagen]

Do you have a good idea how we could test that?

[cbenhagen]

@GroovinChip I mean we can just test if we find the MacosToolbarPassthrough widget. That's easy enough but to test if it really fixes the maximizing issue would be a challenge.

[cbenhagen]

@GroovinChip added some tests in 63777d4.

[GroovinChip]

LGTM