fix: menu: adds onclick handler back to main button when secondarybuttonprops fixing menu onchange

[dkilgore-eightfold]

SUMMARY:

The onClick event was deferred to the ...rest spread via MenuItemButton props, this caused a regression in Menu as the onClick callback was being used internally to trigger Menu onChange callback upstream.

BEFORE FIX:

before.mp4

AFTER FIX:

after.mp4

JIRA TASK (Eightfold Employees Only):

ENG-85650

CHANGE TYPE:

• Bugfix Pull Request
• Feature Pull Request

TEST COVERAGE:

• Tests for this change already exist
• I have added unittests for this change

TEST PLAN:

Pull the PR branch and run yarn and yarn storybook. Verify the Menu stories behave as expected.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.