Skip to content

fix: consume Shift on Enter - #260

Merged
pzauner merged 1 commit into
mainfrom
codex/fix-shift-enter-autocap
Jul 18, 2026
Merged

fix: consume Shift on Enter#260
pzauner merged 1 commit into
mainfrom
codex/fix-shift-enter-autocap

Conversation

@pzauner

@pzauner pzauner commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • consume a manual Shift one-shot when Enter is pressed
  • preserve smart capitalization by allowing it to request a fresh Shift after the newline
  • add service-level and controller regression coverage for both setting states

Root cause

A physical Shift press created a one-shot state that Enter did not consume. With automatic capitalization at line starts disabled, the stale one-shot incorrectly uppercased the next character.

Validation

  • PhysicalKeyboardInputMethodServiceDeviceBehaviorTest
  • TextInputControllerTest
  • ModifierStateControllerTest
  • built and installed 0.86-nightly.20260718.125918 with scripts/build-nightly-debug.sh
  • confirmed on a Titan 2 with automatic capitalization both disabled and enabled

Fixes #241

@pzauner
pzauner marked this pull request as ready for review July 18, 2026 11:10
@pzauner
pzauner merged commit e4e8940 into main Jul 18, 2026
1 check passed
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.

Shift+Enter does not disable shift after enter is pressed

1 participant