Skip to content

Commit 2728901

Browse files
committed
Merge branch 'unite2025/lsp-uitk' of https://github.com/Unity-Technologies/a11y-public-sample into unite2025/lsp-uitk
2 parents 3656603 + ba3a97a commit 2728901

File tree

3 files changed

+7
-11
lines changed

3 files changed

+7
-11
lines changed

Assets/Scripts/Screen Reader/UITk/VisualTreeAccessibilityUpdater.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -644,8 +644,8 @@ void UpdateAccessibilityHierarchyRecursively(VisualElement element,
644644
shouldBeIgnored |= accElement.isIgnored;
645645
modal = accElement.isModal;
646646

647-
// If the branch is visible but the AccessibleElement does not have a node yet then create it.
648-
if (!shouldBeIgnored)
647+
// If the branch is visible but the AccessibleElement does not have a node yet, then create it.
648+
if (!shouldBeIgnored && !modal)
649649
{
650650
if (!IsNodeValid(accElement.node))
651651
{

Assets/Scripts/UITk/Controls/Popup.cs

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -75,14 +75,8 @@ public PopupContent()
7575

7676
PopupOverlay m_Overlay;
7777

78-
ClosePolicy m_ClosePolicy = ClosePolicy.CloseOnPressOutside | ClosePolicy.CloseOnEscape;
79-
8078
[UxmlAttribute, CreateProperty]
81-
public ClosePolicy closePolicy
82-
{
83-
get => m_ClosePolicy;
84-
set => m_ClosePolicy = value;
85-
}
79+
public ClosePolicy closePolicy { get; set; } = ClosePolicy.CloseOnPressOutside | ClosePolicy.CloseOnEscape;
8680

8781
public Popup()
8882
{

Assets/Scripts/UITk/MainView.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -750,7 +750,7 @@ void MoveCard(bool shouldMoveLeft, int count = 1)
750750
// two cards.
751751
m_MainView.schedule.Execute(() =>
752752
AssistiveSupport.notificationDispatcher.SendLayoutChanged(node)
753-
).ExecuteLater(50);
753+
).ExecuteLater(100);
754754

755755
if (moved)
756756
{
@@ -761,7 +761,7 @@ void MoveCard(bool shouldMoveLeft, int count = 1)
761761
// processed first so that the announcement is not interrupted by the focus change.
762762
m_MainView.schedule.Execute(() =>
763763
AssistiveSupport.notificationDispatcher.SendAnnouncement(message)
764-
).ExecuteLater(100);
764+
).ExecuteLater(200);
765765
}
766766

767767
/*var accElement = draggable.transform.GetComponent<AccessibleElement>();
@@ -827,6 +827,8 @@ void ShowGameView(Gameplay.DifficultyLevel level)
827827
void ShowExitGamePopup()
828828
{
829829
m_ExitGamePopup.Show();
830+
831+
AccessibilityManager.GetService<UITkAccessibilityService>()?.RebuildHierarchy();
830832
}
831833

832834
void CloseExitGamePopup()

0 commit comments

Comments
 (0)