From: minho.sun <minho.sun@samsung.com>
Date: Thu, 24 Nov 2016 07:35:37 +0000 (+0900)
Subject: Add focus indicator only when attaching physicalkeyboard
X-Git-Tag: dali_1.2.18~7^2
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=9cb232d3175473cbdcb4729432678bd2e0160007

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 <minho.sun@samsung.com>
---

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() )
     {