From 6e3b52ecbaf68a6e6156a5256e89596c9726e353 Mon Sep 17 00:00:00 2001 From: Seoyeon Kim Date: Fri, 27 Oct 2023 14:35:03 +0900 Subject: [PATCH] [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 --- dali-toolkit/internal/controls/text-controls/text-field-impl.cpp | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) 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(Toolkit::Control::Property::PADDING); -- 2.7.4