From: Artur Świgoń Date: Mon, 8 Mar 2021 13:00:14 +0000 (+0100) Subject: Make AccessibilityManager::SetCurrentFocusActor more usable X-Git-Tag: dali_2.0.19~10^2 X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=fe08faff0b7bbe1f6573b42aa17201c513b793a6;hp=c4ab3a9f3d6ee3b0149feccc78169fc5cb5da2ed Make AccessibilityManager::SetCurrentFocusActor more usable Dali::Accessibility::Accessible::SetCurrentlyHighlightedActor() only sets a private field in the AT-SPI bridge in dali-adaptor, whereas Dali::Toolkit::DevelControl::GrabAccessibilityHighlight() calls Dali::Accessibility::Component::GrabHighlight() which actually attaches a highlight frame to an actor. Change-Id: I8898ff96fdd97898c2bd476e4faa75cbc0574986 --- diff --git a/dali-toolkit/internal/accessibility-manager/accessibility-manager-impl.cpp b/dali-toolkit/internal/accessibility-manager/accessibility-manager-impl.cpp index e3f4209..3a20928 100644 --- a/dali-toolkit/internal/accessibility-manager/accessibility-manager-impl.cpp +++ b/dali-toolkit/internal/accessibility-manager/accessibility-manager-impl.cpp @@ -132,9 +132,7 @@ Actor AccessibilityManager::GetActorByFocusOrder(const unsigned int order) bool AccessibilityManager::SetCurrentFocusActor(Actor actor) { - Dali::Accessibility::Accessible::SetCurrentlyHighlightedActor(actor); - - return true; + return Toolkit::DevelControl::GrabAccessibilityHighlight(actor); } Actor AccessibilityManager::GetCurrentFocusActor()