Skip to content

Conversation

Tsvetomir-Hr
Copy link
Contributor

@Tsvetomir-Hr Tsvetomir-Hr self-assigned this Sep 12, 2025
@Tsvetomir-Hr Tsvetomir-Hr requested a review from a team as a code owner September 12, 2025 08:57
@Tsvetomir-Hr Tsvetomir-Hr added the merge-to-production Use this label to get a comment to choose whether to merge the PR to production label Sep 12, 2025
Copy link
Contributor

Hello @Tsvetomir-Hr,

Check the below option if you would like to automatically generate PR to production. The automation uses the branch for the cherry-pick, and then will delete the branch. Please, do not delete it manually.

  • create PR to production

@@ -28,8 +29,13 @@ To ensure the Dialog closes on `ESC` even after focusing the content, wrap the D
Width="500px"
Height="300px"
Class="focusable-content">
<DialogTitle>
<div tabindex="0" class="dialog-wrapper">
Copy link
Contributor

@dimodi dimodi Sep 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also consider tabindex -1 if you want the element to be focusable with a click, but not take part in the tab navigation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-to-production Use this label to get a comment to choose whether to merge the PR to production
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants