Skip to content

Conversation

@vnbaaij
Copy link
Collaborator

@vnbaaij vnbaaij commented Jan 31, 2024

  • [NavMenu] Abort KeyDown handlers if target id not correct
  • [NavMenu] Bind Expander KeyDown handler to the item instead of the whole menu
    Also:
  • Fix smal MessageBarDemo issue

- [NavMenu] Abort KeyDown handlers if target id not correct
- [NavMenu] Bind Expander KeyDown handler to the item instead of the whole menu
@vnbaaij vnbaaij added this to the V4.3.2 milestone Jan 31, 2024
@vnbaaij vnbaaij requested a review from dvoituron January 31, 2024 11:06
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1414.westeurope.3.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1414.westeurope.3.azurestaticapps.net

@vnbaaij vnbaaij enabled auto-merge (squash) January 31, 2024 13:15
@vnbaaij vnbaaij requested a review from dvoituron January 31, 2024 13:15
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1414.westeurope.3.azurestaticapps.net

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1414.westeurope.3.azurestaticapps.net

@vnbaaij vnbaaij requested a review from dvoituron January 31, 2024 13:46
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-pebble-0dc79cb03-1414.westeurope.3.azurestaticapps.net

@vnbaaij vnbaaij merged commit bbb5e48 into dev Jan 31, 2024
@vnbaaij vnbaaij deleted the users/vnbaaij/navmenu-tweaks branch January 31, 2024 13:54
vnbaaij added a commit that referenced this pull request Jan 31, 2024
* - Fix smal MessageBarDemo issue
- [NavMenu] Abort KeyDown handlers if target id not correct
- [NavMenu] Bind Expander KeyDown handler to the item instead of the whole menu

* Undo some testing work

* Process review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants