From 9cb232d3175473cbdcb4729432678bd2e0160007 Mon Sep 17 00:00:00 2001 From: "minho.sun" Date: Thu, 24 Nov 2016 16:35:37 +0900 Subject: [PATCH 1/1] Add focus indicator only when attaching physicalkeyboard Add focus indicator only when attaching physicalkeyboard. When user wants to preset focus actor before physicalkeyboard is attached, they can call SetCurrentFocus(). Before this patch, it always displays focus indicator whether pyhisicalkeyboard is attached or not. Now, if pyhisicalkeyboard is not attached, SetCurrentFocus() dosen't add focus indicator to focus actor. Change-Id: Iab67127a1079b724f68ab44683684cc06360d82c Signed-off-by: minho.sun --- .../internal/focus-manager/keyboard-focus-manager-impl.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp b/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp index 54d8afa..e6ed691 100644 --- a/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp +++ b/dali-toolkit/internal/focus-manager/keyboard-focus-manager-impl.cpp @@ -149,9 +149,11 @@ bool KeyboardFocusManager::DoSetCurrentFocusActor( const unsigned int actorID ) if( actor && actor.IsKeyboardFocusable() ) { mIsFocusIndicatorEnabled = true; - // Draw the focus indicator upon the focused actor - actor.Add( GetFocusIndicatorActor() ); - + // Draw the focus indicator upon the focused actor when PhysicalKeyboard is attached + if( mIsKeyboardFocusEnabled ) + { + actor.Add( GetFocusIndicatorActor() ); + } // Send notification for the change of focus actor if( !mFocusChangedSignal.Empty() ) { -- 2.7.4