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

Corpus Viewer: Run search when finished typing the query (on enter). #740

Merged
merged 2 commits into from
Nov 14, 2021

Conversation

lanzagar
Copy link
Contributor

Issue

Fixes #704 using Option 1.

Description of changes

Do not run search after every keypress (slow), but only once after typing the whole query and pressing Enter.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec
Copy link
Collaborator

/rebase

1 similar comment
@PrimozGodec
Copy link
Collaborator

/rebase

@PrimozGodec
Copy link
Collaborator

@lanzagar can you rebase on master? We cannot use /rebase yet since rebase includes workflows files. It will work on PRs from the current master.

@codecov-commenter
Copy link

codecov-commenter commented Nov 4, 2021

Codecov Report

Merging #740 (6348468) into master (b99a1c8) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #740   +/-   ##
=======================================
  Coverage   74.35%   74.35%           
=======================================
  Files          73       73           
  Lines        9669     9668    -1     
  Branches     1320     1320           
=======================================
  Hits         7189     7189           
+ Misses       2227     2224    -3     
- Partials      253      255    +2     

@PrimozGodec PrimozGodec merged commit 0839602 into biolab:master Nov 14, 2021
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.

Corpus Viewer: Add delay on search
4 participants