8000 fix: preserve Windows accented input and navigation by rafatahmed · Pull Request #235 · rafatahmed/Flahax · GitHub
[go: up one dir, main page]

Skip to content

fix: preserve Windows accented input and navigation - #235

Merged
rafatahmed merged 1 commit into
mainfrom
fix/windows-unicode-input
Nov 7, 2026
Merged

rafatahmed merged 1 commit into
mainfrom
fix/windows-unicode-input

Conversation

@rafatahmed
Copy link
Copy Markdown
Owner

Summary

Fix Windows flahax mix treating à as an extended-key prefix and consuming the following character. Read native Unicode key events while retaining navigation, editing and held-key repeats.

Windows console event
  -> printable character or navigation key
  -> mix text entry and selection

Closes #127.

Evidence

  • Before: the native hidden-console regression fails on merged main: queued à followed by H does not return à.
    After: the same regression passes. Native records cover à/H/P/K/M text entry, all four arrows, Home/End, Tab/Shift+Tab, Enter, Backspace, Escape, repeats, Alt+numpad release and ignored events. The real mix widgets retain the text and select the expected option.
  • Terminal tests: 29 run, 27 passed and two native POSIX checks skipped on Windows. The changed terminal module has zero missing statements and branches (222 statements, 72 branches).
  • Clean wheel/sdist installs and strict documentation build passed. Style, changelog, release hygiene and compilation checks passed.
  • Full suite: 1,422 discovered invocations reconciled; 1,419 passed, zero failures/errors, three skips. The built-site check subsequently passed; the remaining two checks require native POSIX. Required live PHREEQC checks passed. Source hashes were unchanged throughout the run.
  • Standards/spec reviews found no required changes; one optional test-helper consolidation suggestion remains.

Native tests inject records into a real hidden Windows console. No physical keyboard-layout or interactive Windows Terminal trial is claimed. POSIX input handling is unchanged; native Linux/macOS checks belong to hosted CI.

Merge Danger

Door: Two-way

Blast Radius: Input

The Windows input adapter changes; the legacy stream decoder remains available for compatibility. No solver, formula, catalogue or numerical-tolerance changes.

@rafatahmed rafatahmed added the bug Something isn't working label Nov 7, 2026
@rafatahmed rafatahmed self-assigned this Nov 7, 2026
@read-the-docs-community
Copy link
Copy Markdown

Documentation build overview

📚 Flahax | 🛠️ Build #35718033 | 📁 Comparing 5a9fb6e against latest (c233eda)

  🔍 Preview build  

1 file changed
± changelog.html

@rafatahmed
rafatahmed marked this pull request as ready for review November 7, 2026 13:15
@codecov
codecov Bot commented Nov 7, 2026 •
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (c233eda) to head (5a9fb6e).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #235   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          115       115           
  Lines        13510     13550   +40     
  Branches      2664      2674   +10     
=========================================
+ Hits         13510     13550   +40     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rafatahmed
rafatahmed merged commit 1914034 into main Nov 7, 2026
13 checks passed
@rafatahmed
rafatahmed deleted the fix/windows-unicode-input branch November 7, 2026 13:15
rafatahmed added a commit that referenced this pull request Nov 7, 2026
fix: preserve Windows accented input and navigation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mix TUI on Windows: typing "à" swallows the next key

1 participant

0