[Tizen] Remove the duplicated call with Focused call 53/300553/1 accepted/tizen/7.0/unified/20231030.164720
authorSeoyeon Kim <seoyeon2.kim@samsung.com>
Fri, 27 Oct 2023 05:35:03 +0000 (14:35 +0900)
committerSeoyeon Kim <seoyeon2.kim@samsung.com>
Fri, 27 Oct 2023 05:42:19 +0000 (14:42 +0900)
- 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>
dali-toolkit/internal/controls/text-controls/text-field-impl.cpp

index faad888..3c607be 100644 (file)
@@ -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);