Chris Forrette
Chris Forrette
@zkat Good! Sorry for the radio silence. I've been ironing out different states of searching to get them feeling right, and also being distracted by holiday busy-ness and such. After...
@larsgw No worries, you're not blocking me or anything. Let me know if you get started on it, and I'll do the same if I get to it first. Thanks!
@larsgw Awesome, thanks!
@zkat OK, I think this is ready for another look! I think I ironed out the input focus stuff—I did a few tests with and without an initial term search,...
@zkat Awesome, thanks! For getting this merged, do you want the table display and the one-liner package description in place first?
@zkat No worries! I'll implement the `PackageSearchResult` component from @larsgw for the package summary once npm/tink#38 is merged.
@zkat Btw, is there a real package installer I should use yet? I noticed [`lib/installer.js`](https://github.com/npm/tink/blob/latest/lib/installer.js) but it looks like that's installing everything from `package.json`. Should I add an `addPackage` method...
@SchneiderOr I was waiting on #38 to implement here, I'll go nudge over there...
@zkat Also I'm not sure if you saw my last comment, but is there a real install function I should wire up to this? Or should it remain nerfed?
@larsgw Hey there, I'm waiting on this to land to wrap up #11. Aside from fixing the build and updating from master are there any other hold-ups to merging this?