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

Implement search #4

Open
jonathanKingston opened this issue May 15, 2017 · 6 comments
Open

Implement search #4

jonathanKingston opened this issue May 15, 2017 · 6 comments
Assignees
Milestone

Comments

@jonathanKingston
Copy link
Owner

Search could be controlled through a shortcut to open the sidebar and focus a search box.

Filtering and opening all relevant container panels whilst highlighting them.

@grahamperrin
Copy link

grahamperrin commented May 27, 2017

… shortcut …

The usual: beware of conflicts. Consider the shortlist of points at https://github.com/mozilla/testpilot-containers/issues/245#issuecomment-292862635

I'd prefer a method of presenting the search interface that allows the user to assign a shortcut, without assigning a shortcut by default.

… Filtering and opening all relevant container panels …

Will titles and URLs of pages be matched? (Anything more, or less?)

jonathanKingston added a commit that referenced this issue Jun 7, 2017
@jonathanKingston
Copy link
Owner Author

Will titles and URLs of pages be matched? (Anything more, or less?)

Both. Searching in page etc will be super buggy slow and noisy.

The sidebar will have Ctrl+Shift+Period to open, I'm going to make search the default when this lands.

@jonathanKingston
Copy link
Owner Author

@grahamperrin I just pushed a work in progress patch. It filters fine, however this isn't super useful alone.

Required is:

  • Keyboard navigation to go through the items easily
  • Auto expand all matching containers
  • Esc key / perhaps other to cancel
  • UI improvements for icons and position
  • Auto focus on open

@jonathanKingston
Copy link
Owner Author

The logic used is:

  • Split each term used "keyword keyword"
  • Lowercase each keyword
  • Match url and title against keyword
  • If all keyword matches then tab is shown

@grahamperrin
Copy link

… a work in progress patch …

Thanks, I don't see the feature, am I missing something?

NB for me this is theoretical:

  • I wouldn't actually use a search feature in Sea containers
  • I treat Sea containers as complementary to extensions such as Conex.

@tradewatcher
Copy link

The option to show a filter textbar at the top of sidebar is an idea but for me it would be useless. Adress bar have a great search function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants