Skip to content

fix: disable fullscreen on F while typing (#1441)#1454

Merged
manojVivek merged 2 commits intoresponsively-org:mainfrom
bhnprksh222:f-key-issue
Dec 16, 2025
Merged

fix: disable fullscreen on F while typing (#1441)#1454
manojVivek merged 2 commits intoresponsively-org:mainfrom
bhnprksh222:f-key-issue

Conversation

@bhnprksh222
Copy link
Contributor

@bhnprksh222 bhnprksh222 commented Dec 16, 2025

Fix: Prevent fullscreen toggle when typing in input fields

This PR resolves an issue where pressing F while typing inside an input field (input, textarea, or contentEditable) incorrectly triggered fullscreen mode in the webview.

A guard has been added to skip the fullscreen shortcut when a text-entry element is focused.

Fixes

Fixes #1441

Summary of Changes

  • Added isUserTyping() check in preload-webview.ts
  • Prevent fullscreen toggle when focused inside an input or editable element
  • Fullscreen behavior unchanged in all other contexts

@CLAassistant
Copy link

CLAassistant commented Dec 16, 2025

CLA assistant check
All committers have signed the CLA.

@manojVivek
Copy link
Collaborator

@bhnprksh222 Thanks for the contribution, the code looks good, can you please attach a screen recording showing the fix working?
Showing that the full screen switch works when not typing and prevented when typing?

@bhnprksh222
Copy link
Contributor Author

f-key-issue-fix-ezgif com-video-to-gif-converter (1)

FYI

@bhnprksh222
Copy link
Contributor Author

@manojVivek Could you please refer to the uploaded video?

Copy link
Collaborator

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great, thank you so much for this contribution!

@manojVivek manojVivek merged commit bf1993b into responsively-org:main Dec 16, 2025
4 checks passed
@manojVivek
Copy link
Collaborator

@all-contributors Please add @bhnprksh222 for code.

@allcontributors
Copy link
Contributor

@manojVivek

I've put up a pull request to add @bhnprksh222! 🎉

@fmorrisey
Copy link

@bhnprksh222 This issue still persists in v1.17.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Typing 'F' in a text field attempts to full screen the page

4 participants