Skip to content

fix: DH-14237: down arrow in console not returning to blank field#1082

Merged
mattrunyon merged 1 commit intodeephaven:mainfrom
mattrunyon:console-down-arrow-fix
Feb 9, 2023
Merged

fix: DH-14237: down arrow in console not returning to blank field#1082
mattrunyon merged 1 commit intodeephaven:mainfrom
mattrunyon:console-down-arrow-fix

Conversation

@mattrunyon
Copy link
Copy Markdown
Collaborator

Fixes DH-14237

Was incorrectly converted in #646

Switched the arrow keys to use code instead of keyCode because RTL user-event does not set keyCode which is a deprecated property

@mattrunyon mattrunyon added the bug Something isn't working label Feb 9, 2023
@mattrunyon mattrunyon requested a review from mofojed February 9, 2023 18:41
@mattrunyon mattrunyon self-assigned this Feb 9, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 9, 2023

Codecov Report

Merging #1082 (8c08db8) into main (447421f) will increase coverage by 0.24%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1082      +/-   ##
==========================================
+ Coverage   41.86%   42.10%   +0.24%     
==========================================
  Files         432      432              
  Lines       32555    32555              
  Branches     8191     8191              
==========================================
+ Hits        13628    13707      +79     
+ Misses      18874    18795      -79     
  Partials       53       53              
Flag Coverage Δ
unit 42.10% <100.00%> (+0.24%) ⬆️

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

Impacted Files Coverage Δ
packages/console/src/ConsoleInput.tsx 67.69% <100.00%> (+30.08%) ⬆️
packages/console/src/monaco/MonacoUtils.ts 57.39% <0.00%> (+9.56%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mattrunyon mattrunyon merged commit e15c125 into deephaven:main Feb 9, 2023
@mattrunyon mattrunyon deleted the console-down-arrow-fix branch February 9, 2023 19:40
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants