Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add UX to cycle through completions in the REPL #2463

Merged
merged 27 commits into from
Jul 1, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
989e2b4
feat: add UX to cycle through completions
Snehil-Shah Jun 26, 2024
1d6ad40
Apply suggestions from code review
kgryte Jun 26, 2024
e8e0e9f
Apply suggestions from code review
Snehil-Shah Jun 27, 2024
535986f
fix: avoid navigating hidden completions
Snehil-Shah Jun 28, 2024
4babb98
refactor: use `\n` instead of `\r\n`
Snehil-Shah Jun 28, 2024
755ddee
fix: abnormal behavior with ENTER
Snehil-Shah Jun 28, 2024
2fcb356
fix: duplicate line events
Snehil-Shah Jun 28, 2024
9de549d
fix: incorrect height
Snehil-Shah Jun 28, 2024
2b1f666
fix: bug with ENTER
Snehil-Shah Jun 28, 2024
7510ba9
test: add test case
Snehil-Shah Jun 28, 2024
da8f7ee
test: fix brittle test cases
Snehil-Shah Jun 28, 2024
63ce461
feat: allow sustaining the completions panel
Snehil-Shah Jun 29, 2024
d4afb81
fix: reorder keypress handlers according to precedence
Snehil-Shah Jun 29, 2024
858eddb
fix: make it receptive to SIGINT events
Snehil-Shah Jun 29, 2024
6fd48a8
fix: reorder keypress handlers according to precedence
Snehil-Shah Jun 29, 2024
5bb319c
feat: allow navigating the line when navigating
Snehil-Shah Jun 29, 2024
7f4eac7
refactor: clean logic and avoid unnecessary re-rendering
Snehil-Shah Jun 29, 2024
ddcd3d6
test: remove outdated test
Snehil-Shah Jun 29, 2024
494f845
fix: don't close when completer is already closed
Snehil-Shah Jun 29, 2024
141bfe4
test: add test case for live completions
Snehil-Shah Jun 30, 2024
5410b2b
fix: incorrect logic
Snehil-Shah Jul 1, 2024
3677b2a
fix: incorrect keypress handlers
Snehil-Shah Jul 1, 2024
e53f691
test: update test names
Snehil-Shah Jul 1, 2024
4b8a752
docs: fix jsdoc
Snehil-Shah Jul 1, 2024
71b5c4c
fix: update after inserting a completion
Snehil-Shah Jul 1, 2024
e862980
Apply suggestions from code review
kgryte Jul 1, 2024
97329bf
Apply suggestions from code review
kgryte Jul 1, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion lib/node_modules/@stdlib/repl/lib/completer.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@

// VARIABLES //

var debug = logger( 'repl:completer' );
var debug = logger( 'repl:completer:callback' );


// FUNCTIONS //
Expand All @@ -66,7 +66,7 @@
}
list = objectKeys( hash );

// TODO: sort such that printed columns are in lexicographic order, not rows, similar to bash behavior!

Check warning on line 69 in lib/node_modules/@stdlib/repl/lib/completer.js

View workflow job for this annotation

GitHub Actions / Lint Changed Files

Unexpected 'todo' comment: 'TODO: sort such that printed columns are...'

// Sort the values in lexicographic order:
list = list.sort();
Expand Down
Loading
Loading