Skip to content
This repository has been archived by the owner. It is now read-only.

[ProjectSearch] tracking issue #1108

Closed
jasonLaster opened this issue Nov 4, 2016 · 11 comments
Closed

[ProjectSearch] tracking issue #1108

jasonLaster opened this issue Nov 4, 2016 · 11 comments

Comments

@jasonLaster
Copy link
Contributor

@jasonLaster jasonLaster commented Nov 4, 2016

We'd like to re-add the ability to search across the project.

Links

Steps:

  • add a UI for showing project search #3090
  • Add a component for toggling project search between text and files #3091
  • add a text search function
  • Add a redux action for loading source texts #3331
  • Move search functionality to redux #3342
  • Refactor Project Search to support text search
  • Make source search stream #3270
  • Create a "search" tab #3279
  • Cleanup the css and styles for dark theme #3326
  • Auto focus on the first result in the list
  • Highlight line where the text match was found when source file is opened #3467
  • Add keyboard actions for accessibility #3501
  • should scroll to keep items in focus
  • File path does not appear in nightly #3782
  • Improve Text Search -> Source Transition #3787
  • Flow type the UI - I started here, but struggled with ManagedTree.
@jasonLaster jasonLaster changed the title [Project Search] [Project Search] Tracking Ticket Nov 5, 2016
@mmucci mmucci added this to the Release 54 milestone Jan 24, 2017
@mmucci mmucci removed this from the Release 54 milestone Jan 24, 2017
@jasonLaster jasonLaster changed the title [Project Search] Tracking Ticket Project Search Jan 27, 2017
@clarkbw
Copy link
Contributor

@clarkbw clarkbw commented Mar 12, 2017

Pulling this image out so you don't have to jump to the doc.

screen shot 2017-03-12 at 12 03 40 pm

@wldcordeiro
Copy link
Contributor

@wldcordeiro wldcordeiro commented Mar 12, 2017

I'd love to get hacking on this. What/where would I need to work to do these parts?

add a debugger server endpoint
add debugger client + command methods

@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Mar 12, 2017

@clarkbw
Copy link
Contributor

@clarkbw clarkbw commented May 5, 2017

@wanderview really wants this 😄

@clarkbw clarkbw added this to Debugger in Web Compatibility May 5, 2017
@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Jun 3, 2017

I started hacking on this earlier today and made some progress

I'll post a couple PRs based on the work so that it doesn't go stale soon

  • add a source search function
  • Refactor Project Search to support text search
  • Add a component for toggling project search between text and files
  • Add a redux action for loading source texts

I started one and two and punted on three and four naturally :)

@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Jun 5, 2017

I updated the issue description to have a good checklist

@jasonLaster jasonLaster added this to the July 11th milestone Jun 22, 2017
@jasonLaster jasonLaster modified the milestones: July 25th, July 11th Jul 11, 2017
@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Jul 21, 2017

  • the results should scroll
  • the selection highlight is too strong in dark theme
  • we should exclude watch expressions (sources should have a URL)
@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Jul 25, 2017

Here is a patch for styling project search:
https://gist.github.com/jasonLaster/b02fa35094cf5ef3d5055d329408aefc

@jasonLaster jasonLaster changed the title Project Search [ProjectSearch] tracking issue Jul 31, 2017
@jasonLaster jasonLaster added this to the August 15th milestone Aug 1, 2017
@jasonLaster jasonLaster removed this from the August 15th milestone Aug 15, 2017
@bughit
Copy link

@bughit bughit commented Aug 24, 2017

which firefox version is this planned for?

@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Aug 24, 2017

It's turned on nightly today :)

@jasonLaster
Copy link
Contributor Author

@jasonLaster jasonLaster commented Sep 22, 2017

going to close this as we have done the big items and shipped!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants