From: Seoyeon Kim <seoyeon2.kim@samsung.com>
Date: Fri, 27 Oct 2023 05:35:03 +0000 (+0900)
Subject: [Tizen] Remove the duplicated call with Focused call
X-Git-Tag: accepted/tizen/7.0/unified/20231030.164720^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6e3b52ecbaf68a6e6156a5256e89596c9726e353;p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git

[Tizen] Remove the duplicated call with Focused call

- To call `Activate()` of IME is already called in `OnKeyInputFocusGained()`
 and `OnKeyInputFocusGained()` is called by `OnTap()`.
- So, IME activate call is called twice unnecessarily.

- Also, IME properties did not be set in `OnTap`,
  so the normal layout of input panel shows first,
 and then another type of input panel shows again. (Issue)
- Fix this issue to erase to call `Activate` in `OnTap()`.

Change-Id: Ie18b35042b0b7878efd0a8fe3fbbd5da1d3e6def
Signed-off-by: Seoyeon Kim <seoyeon2.kim@samsung.com>
---

diff --git a/dali-toolkit/internal/controls/text-controls/text-field-impl.cpp b/dali-toolkit/internal/controls/text-controls/text-field-impl.cpp
index faad888..3c607be 100644
--- a/dali-toolkit/internal/controls/text-controls/text-field-impl.cpp
+++ b/dali-toolkit/internal/controls/text-controls/text-field-impl.cpp
@@ -815,10 +815,7 @@ bool TextField::OnAccessibilityActivated()
 void TextField::OnTap(const TapGesture& gesture)
 {
   DALI_LOG_INFO(gTextFieldLogFilter, Debug::Verbose, "TextField::OnTap %p\n", mController.Get());
-  if(mInputMethodContext && IsEditable())
-  {
-    mInputMethodContext.Activate();
-  }
+
   // Deliver the tap before the focus event to controller; this allows us to detect when focus is gained due to tap-gestures
   Extents padding;
   padding                   = Self().GetProperty<Extents>(Toolkit::Control::Property::PADDING);