Skip to content

🤖 fix: disable native spellchecker in Electron windows#3413

Merged
ammario merged 1 commit into
mainfrom
spell-check-rrem
May 28, 2026
Merged

🤖 fix: disable native spellchecker in Electron windows#3413
ammario merged 1 commit into
mainfrom
spell-check-rrem

Conversation

@ammar-agent

@ammar-agent ammar-agent commented May 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

Disable Electron's native spellchecker across the renderer windows. mux is a coding tool whose text inputs are full of code, file paths, and identifiers, so the OS spellchecker just produces noisy red squiggles with no value.

Implementation

Added spellcheck: false to webPreferences for:

  • The main app window (src/desktop/main.ts) — where all chat/text input happens.
  • The popped-out terminal window (src/desktop/terminalWindowManager.ts) — content is code.

The splash window has no text inputs, so it was left untouched. Per-element spellCheck={false} attributes that already existed on individual React inputs remain (now redundant but harmless).

Validation

  • make static-check passes (eslint, typecheck for both app + main tsconfig projects, formatting, docs links).

Generated with mux • Model: anthropic:claude-opus-4-8 • Thinking: xhigh • Cost: $0.43

@ammar-agent

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@ammario ammario merged commit 4912ad6 into main May 28, 2026
24 checks passed
@ammario ammario deleted the spell-check-rrem branch May 28, 2026 20:38
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.

2 participants