Change useMenuTrigger onPressStart to state.open

[subvertallchris]

Closes #5988

✅ Pull Request Checklist:

• Link to issue Select menu closes unexpectedly while holding mouse button and cursor leaving then returning #5988
• Added/updated unit tests and storybook for this change (for new code or code which already has tests).
• Filled out test instructions.
• [NA] Updated documentation (if it already exists for this component).
• [NA] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

1. Review the problem as described at Select menu closes unexpectedly while holding mouse button and cursor leaving then returning #5988
2. Open Storybook and browse to MenuTrigger
3. Try to replicate the bug as originally described

I tried to write a new test covering the original condition and failed. I don't have the fluency in react-aria to know how to simulate the behavior. A test did cover the basic behavior, though, and it was updated to reflect the use of open instead of toggle.

[subvertallchris]

@LFDanLu thanks for the advice and recommendation!

My CLA is signed, FYI.

[LFDanLu]

@subvertallchris Thanks for the speedy turnaround! I'll see if I can figure out a way to test this flow when I get some spare time

[subvertallchris]

Happy to help! Do you think more tests are necessary prior to merge or is this mergeable as is now?

[LFDanLu]

I'd prefer having a test that mimics the user flow of pointer down on the trigger button + move off the button + move back on the button and confirms that the open state doesn't change using a flow similar to this, but I personally wouldn't make it a hard requirement for this PR since it might be a bit tricky with the coordinate mocking. Definitely will try to get this into a general upcoming testing session this Monday to see if we can spot any cross browser/device edge cases.

[LFDanLu]

GET_BUILD

[rspbot]

Build successful! 🎉

• View the storybook
• View the storybook-17
• View the storybook-16
• View the documentation

[snowystinger]

Thanks for the PR!

[reidbarber]

GET_BUILD

[rspbot]

Build successful! 🎉

• View the storybook
• View the storybook-17
• View the storybook-16
• View the documentation

[rspbot]

## API Changes

unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any', access: 'private' }
unknown top level export { type: 'any', access: 'private' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'identifier', name: 'Column' }
unknown top level export { type: 'identifier', name: 'Column' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown type { type: 'link' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }
unknown top level export { type: 'any' }

[reidbarber]

LGTM, thanks!