window.open prevents editable input/textarea to hide keyboard on blur
Reported by
juw...@gmail.com,
Dec 30 2016
|
|||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0 Steps to reproduce the problem: 1. Open any page that includes as example onclick window.open handler 2. set focus inside any editable field 3. and click on link that calls window.open (the problem is even more deep, calling blur() on editable element before window.open does not hide keyboard.) What is the expected behavior? The keyboard should be autohided since input field looses focus on new window/tab/popup. What went wrong? Actually on new tab the keyboard can't be hidden by touching screen instead I have to press physical button (right bottom) Did this work before? N/A Does this work in other browsers? Yes Chrome version: Chrome mobile 55-57 Channel: n/a OS Version: 10.0 Flash Version: Shockwave Flash 24.0 r0 On other browsers or on old Chrome it worked as expected before. Issue is reproducable on Android mobile Chrome 55, 56 and 57
,
Jan 4 2017
,
Jan 4 2017
,
Jan 5 2017
,
Jan 5 2017
,
Jan 10 2017
This is likely regressed by https://codereview.chromium.org/2290133002 . assigning yabinh@.
,
Jan 10 2017
,
Jan 10 2017
,
Feb 1 2017
Hello, any update on issue?
,
Feb 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/44b7e0d7c92d708fdc64cb2357bbb2a712551b94 commit 44b7e0d7c92d708fdc64cb2357bbb2a712551b94 Author: yabinh <yabinh@chromium.org> Date: Wed Feb 22 09:14:38 2017 Make chrome hide keyboard when opening a new window We should hide keyboard when opening a new window in Chrome. The CL https://codereview.chromium.org/2290133002 moves the hidding logic from onViewFocusChanged() to onViewDetachedFromWindow() and onAttach(). But the view will not be active when it is detached from window, so the keyboard will not be hidden. To fix that, this CL brings back the hidding logic to onViewFocusChanged(). Thus, we are able to hide keyboard before the view becomes inactive. Since we don’t want to change the current behavior of Webview, i.e., not hiding keyboard when losing focus, so we pass a parameter to distinguish that. To avoid possible regressions, this CL keeps the previous hiding logic in onViewDetachedFromWindow() and onAttach(). Alternatively, we can use a tab observer to determine when to hide keyboard, but it will increase code coupling, so we stick to the above method. BUG= 677646 Review-Url: https://codereview.chromium.org/2709633002 Cr-Commit-Position: refs/heads/master@{#451950} [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/android_webview/java/src/org/chromium/android_webview/AwContents.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/chrome/test/data/android/tabstest/tabs_test.html [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/content/public/android/java/src/org/chromium/content/browser/ContentView.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/content/public/android/java/src/org/chromium/content/browser/input/ImeAdapter.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/content/public/android/javatests/src/org/chromium/content/browser/ContentViewCoreSelectionTest.java [modify] https://crrev.com/44b7e0d7c92d708fdc64cb2357bbb2a712551b94/content/public/android/javatests/src/org/chromium/content/browser/PopupZoomerTest.java
,
Feb 28 2017
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by juw...@gmail.com
, Dec 30 2016569 bytes
569 bytes View Download