Skip to content

chore(web-components): add test for complex focus management#32009

Merged
davatron5000 merged 3 commits into
microsoft:masterfrom
davatron5000:fix/focus-management-on-complex-menu
Jul 24, 2024
Merged

chore(web-components): add test for complex focus management#32009
davatron5000 merged 3 commits into
microsoft:masterfrom
davatron5000:fix/focus-management-on-complex-menu

Conversation

@davatron5000

Copy link
Copy Markdown
Contributor

Previous Behavior

No tests for handling focus on complex nested menus. This is easy to break when refactoring so it's helpful to have a more complex example of focus behavior in the test suite.

New Behavior

Adds fixture for complex submenus to Menu and test for focus state is returned to first item in parent menu

Related Issue(s)

  • Fixes #

@davatron5000
davatron5000 requested a review from a team as a code owner July 15, 2024 23:40
@fabricteam

fabricteam commented Jul 15, 2024

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

✅ No changes found

@fabricteam

Copy link
Copy Markdown
Collaborator

🕵 fluentui-web-components-v3 No visual regressions between this PR and main

@davatron5000
davatron5000 merged commit 380f9c3 into microsoft:master Jul 24, 2024
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jul 25, 2024
* master: (77 commits)
  chore(react-examples): replace storybook deprecated api with static stories (microsoft#32074)
  [chore]: create base class for accordion item and remove style and layout specific api (microsoft#32102)
  release: applying package updates - web-components
  chore(web-components): add test for complex focus management (microsoft#32009)
  [chore]: create base class for avatar and remove style and layout specific api (microsoft#32083)
  [chore]: create base class for text input and remove style and layout specific api (microsoft#32080)
  release: applying package updates - web-components
  chore:(react-nav-preview) Scaffold AppItem (microsoft#32088)
  docs(`react-teaching-popover`): Adding subcomponents' API to `TeachingPopover` documentation page (microsoft#32084)
  feat(web-components): relax setTheme() argument type to allow custom tokens (microsoft#32087)
  release: applying package updates - react-components
  [Chore]: Create Spinner base class to abstract out style and layout specific api. (microsoft#32067)
  [Chore]: Create Progress Bar base class to abstract out style and layout specific api. (microsoft#32066)
  fix(react-storybook-addon): transform decorator to function in withAriaLive() (microsoft#32011)
  fix(motion): improve Web Animations API detection in tests (microsoft#32029)
  chore(eslint-plugin): removes type dependency on @fluentui/react-utilities internals in ban-instanceof-html-element rule (microsoft#32072)
  release: applying package updates - react v8
  release: applying package updates - web-components
  Chore: Create Divider base class to abstract out style and layout specific api (microsoft#32065)
  fix(TimePicker): Clear text when date value changes to null (microsoft#31626)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants