Skip to content

feat: add on_key_event prop to TextInput#183

Open
ccbrown wants to merge 1 commit intomainfrom
text-input-on-key-event
Open

feat: add on_key_event prop to TextInput#183
ccbrown wants to merge 1 commit intomainfrom
text-input-on-key-event

Conversation

@ccbrown
Copy link
Copy Markdown
Owner

@ccbrown ccbrown commented Mar 30, 2026

What It Does

Adds an on_key_event prop to TextInput. This provides a more elegant way to implement custom key binds or disable or override the default ones.

Going to let this PR marinate for a bit before merging.

Related Issues

#97

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 30, 2026

Codecov Report

❌ Patch coverage is 62.50000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.30%. Comparing base (fcbb0ee) to head (7e864d6).

Files with missing lines Patch % Lines
packages/iocraft/src/components/text_input.rs 58.33% 4 Missing and 1 partial ⚠️
packages/iocraft/src/handler.rs 75.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
- Coverage   87.36%   87.30%   -0.06%     
==========================================
  Files          34       34              
  Lines        5358     5366       +8     
  Branches     5358     5366       +8     
==========================================
+ Hits         4681     4685       +4     
- Misses        568      572       +4     
  Partials      109      109              
Files with missing lines Coverage Δ
packages/iocraft/src/handler.rs 90.32% <75.00%> (ø)
packages/iocraft/src/components/text_input.rs 68.45% <58.33%> (-0.37%) ⬇️

Impacted file tree graph

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

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.

1 participant