Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: on-screen-keyboard hides on input blurred in webview #35921

Conversation

kyrylo-hrechykhin
Copy link
Contributor

@kyrylo-hrechykhin kyrylo-hrechykhin commented Oct 5, 2022

Description of Change

Adding patch that fixes on-screen-keyboard (OSK) not hiding on input rendered in webview is blurred. The issue is chromium based (mentioned in the patch) , however it is not clear how to fix it. Changes introduced by this patch fix issue in at least one user case -- where input is blurred due to something else inside the same webview is focused. Changes do not cover the case where input in the webview is blurred due to something else is focused outside of webview. Technically in this case input inside the webview won't be blurred at all due to implementation details of webview.

Checklist

Release Notes

Notes: Fixed on-screen-keyboard not hiding on input rendered in webview is blurred.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Oct 5, 2022
@kyrylo-hrechykhin kyrylo-hrechykhin force-pushed the fix/34300-osk-does-not-hide-on-input-in-webview-blurred branch 2 times, most recently from eddd00b to d0a3be8 Compare October 6, 2022 16:55
@kyrylo-hrechykhin kyrylo-hrechykhin marked this pull request as ready for review October 7, 2022 08:12
@kyrylo-hrechykhin kyrylo-hrechykhin requested a review from a team as a code owner October 7, 2022 08:12
@zcbenz zcbenz added semver/patch backwards-compatible bug fixes target/20-x-y target/21-x-y PR should also be added to the "21-x-y" branch. target/22-x-y PR should also be added to the "22-x-y" branch. component/webview labels Oct 11, 2022
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Oct 11, 2022
@zcbenz zcbenz force-pushed the fix/34300-osk-does-not-hide-on-input-in-webview-blurred branch from d0a3be8 to 56be201 Compare October 11, 2022 10:32
@kyrylo-hrechykhin
Copy link
Contributor Author

kyrylo-hrechykhin commented Oct 11, 2022

Kindly asking contributors to review/merge this PR.
CC: @zcbenz, @codebytere, @deepak1556, @nornagon

@kyrylo-hrechykhin kyrylo-hrechykhin force-pushed the fix/34300-osk-does-not-hide-on-input-in-webview-blurred branch from 56be201 to 77fe69e Compare October 11, 2022 14:10
@jkleinsc jkleinsc merged commit c2cb97e into electron:main Oct 11, 2022
@welcome
Copy link

welcome bot commented Oct 11, 2022

Congrats on merging your first pull request! 🎉🎉🎉

@release-clerk
Copy link

release-clerk bot commented Oct 11, 2022

Release Notes Persisted

Fixed on-screen-keyboard not hiding on input rendered in webview is blurred.

@trop
Copy link
Contributor

trop bot commented Oct 11, 2022

I have automatically backported this PR to "20-x-y", please check out #35981

@trop
Copy link
Contributor

trop bot commented Oct 11, 2022

I have automatically backported this PR to "21-x-y", please check out #35982

@trop
Copy link
Contributor

trop bot commented Oct 11, 2022

I have automatically backported this PR to "22-x-y", please check out #35983

@trop trop bot removed the target/21-x-y PR should also be added to the "21-x-y" branch. label Oct 11, 2022
@trop trop bot added in-flight/22-x-y merged/22-x-y PR was merged to the "22-x-y" branch. merged/21-x-y PR was merged to the "21-x-y" branch. and removed target/22-x-y PR should also be added to the "22-x-y" branch. in-flight/22-x-y labels Oct 11, 2022
khalwa pushed a commit to solarwindscloud/electron that referenced this pull request Feb 22, 2023
…5921)

Co-authored-by: Kyrylo Hrechykhin <khrechykhin@microsoft.com>
zcbenz added a commit that referenced this pull request Jan 26, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.
zcbenz added a commit that referenced this pull request Jan 26, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.
zcbenz added a commit that referenced this pull request Jan 26, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.
zcbenz added a commit that referenced this pull request Jan 29, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.
trop bot added a commit that referenced this pull request Jan 29, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.

Co-authored-by: Cheng Zhao <zcbenz@gmail.com>
codebytere pushed a commit that referenced this pull request Jan 29, 2024
This is a follow up to #35921
that, it fixes more corner cases that on-screen-keyboard does not hide
for webviews.

This change has been applied in Teams for quite a while and should be
reliable enough to introduce to Electron.

Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com>
Co-authored-by: Cheng Zhao <zcbenz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/webview merged/21-x-y PR was merged to the "21-x-y" branch. merged/22-x-y PR was merged to the "22-x-y" branch. semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants