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

Allow query string selection for multiple omniboxes #89

Merged
merged 2 commits into from
May 11, 2020

Conversation

JoshStern
Copy link
Contributor

@JoshStern JoshStern commented May 11, 2020

  • Alter choose check to allow query string selection. Possibly a breaking
    change but we would like to make this behavior default instead of opt-in
    because generally multiple omniboxes without requireMatch would like to
    receive the query on enter.
  • Update dependencies to latest minor versions
  • Fix linting rule
  • Add hnvmrc file

@JoshStern JoshStern requested a review from a team May 11, 2020 15:51
@JoshStern JoshStern force-pushed the jpas/update-deps-and-multi-behavior branch from f1f34e8 to e3ec687 Compare May 11, 2020 17:11
Josh Stern added 2 commits May 11, 2020 13:20
- Update dependencies to latest minor versions
- Fix linting rule
- Add hnvmrc file
Alter choose check to allow query string selection. Possibly a breaking
change but we would like to make this behavior default instead of opt-in
because generally `multipl` omniboxes without `requireMatch` would like
to receive the query on enter.
@JoshStern JoshStern force-pushed the jpas/update-deps-and-multi-behavior branch from e3ec687 to 61cbd80 Compare May 11, 2020 17:20
@JoshStern JoshStern merged commit 456d760 into master May 11, 2020
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.

2 participants