[GH-ISSUE #481] onKeyPress type exists but is not implemented in Solid #124

New issue

Open

opened 2026-03-02 23:44:44 +03:00 by kerem · 2 comments

kerem commented

2026-03-02 23:44:44 +03:00

Owner

Originally created by @GreyElaina on GitHub (Jan 7, 2026).
Original GitHub issue: https://github.com/anomalyco/opentui/issues/481

Originally assigned to: @Adictya on GitHub.

Hey! I noticed that onKeyPress is defined in the Solid types but doesn't actually do anything.

Where it's defined:

packages/solid/src/types/elements.ts line 130 - TextareaProps has onKeyPress?: (event: KeyEvent) => void

What's missing:

packages/solid/src/reconciler.ts has no handling for onKeyPress
Core's Renderable only has onKeyDown, there's no onKeyPress equivalent

So if someone tries to use onKeyPress on a textarea, TypeScript will be happy but the handler will never fire.

Options:

Remove the type definition (breaking change, but honest)
Implement it in the reconciler (would need core support first)
Add a deprecation note

Originally created by @GreyElaina on GitHub (Jan 7, 2026). Original GitHub issue: https://github.com/anomalyco/opentui/issues/481 Originally assigned to: @Adictya on GitHub. Hey! I noticed that `onKeyPress` is defined in the Solid types but doesn't actually do anything. **Where it's defined:** - `packages/solid/src/types/elements.ts` line 130 - `TextareaProps` has `onKeyPress?: (event: KeyEvent) => void` **What's missing:** - `packages/solid/src/reconciler.ts` has no handling for `onKeyPress` - Core's `Renderable` only has `onKeyDown`, there's no `onKeyPress` equivalent So if someone tries to use `onKeyPress` on a textarea, TypeScript will be happy but the handler will never fire. Options: 1. Remove the type definition (breaking change, but honest) 2. Implement it in the reconciler (would need core support first) 3. Add a deprecation note

kerem added the

bug

solid

labels

2026-03-02 23:44:44 +03:00

kerem commented

2026-03-02 23:44:45 +03:00

Author

Owner

@kommander commented on GitHub (Jan 26, 2026):

cc @Adictya thoughts?

@kommander commented on GitHub (Jan 26, 2026): cc @Adictya thoughts?

kerem commented

2026-03-02 23:44:45 +03:00

Author

Owner

@Adictya commented on GitHub (Jan 27, 2026):

Yeah looks like its outdated, looking into it more.

Ill try to propose a longterm solution here and maybe try catching stuff like this in ci. Also i think its time to pull the types to a shared lib ( @msmps lmk if you have any thoughts on this ) will try and put a pr up so we can get started on that too

@Adictya commented on GitHub (Jan 27, 2026): Yeah looks like its outdated, looking into it more. Ill try to propose a longterm solution here and maybe try catching stuff like this in ci. Also i think its time to pull the types to a shared lib ( @msmps lmk if you have any thoughts on this ) will try and put a pr up so we can get started on that too

kerem referenced this issue

2026-03-02 23:45:53 +03:00

[PR #124] [MERGED] Feat: extend for vue #308

kerem referenced this issue

2026-03-02 23:45:54 +03:00

[PR #129] [MERGED] Feat: scroll box (vue) #311

kerem referenced this issue

2026-03-14 09:17:49 +03:00

[PR #124] [MERGED] Feat: extend for vue #1091

kerem referenced this issue

2026-03-14 09:18:05 +03:00

[PR #129] [MERGED] Feat: scroll box (vue) #1093