Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a critical usability issue by refining the logic that determines if an HTML element is a text field. By making this distinction more precise, it restores expected behavior for keyboard shortcuts like undo/redo, which were previously inhibited by non-text input elements, significantly improving the user experience for form interactions. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request fixes an issue where keyboard shortcuts were incorrectly blocked for non-text input fields like checkboxes. The change correctly refines targetIsTextField to only identify text-entry-related input types. The implementation is clear and effectively resolves the bug. I've added one suggestion to slightly optimize the new logic for better performance.
Screen.Recording.2026-03-24.at.9.23.40.AM.movWorking fine after the recent rebase against main and opening up this new PR |
|
@Keavon , here is the new PR, wasn't able to push to the previous one |
Replaces #3866 (branch was force-pushed after file location changed in master)
Bug: Keyboard shortcuts like Cmd+Z were blocked when a checkbox had focus because targetIsTextField() treated all elements as text fields.
Fix: Only treat text-like input types (text, password, email, etc.) as text fields, not checkboxes or radio buttons.
Screen.Recording.2026-03-07.at.5.32.57.PM.mov
Fixes: https://discord.com/channels/731730685944922173/731738914812854303/1479627959127117924