Skip to content
This repository has been archived by the owner on Jan 5, 2019. It is now read-only.

Add search results table #109

Merged
merged 23 commits into from
Jan 17, 2017
Merged

Add search results table #109

merged 23 commits into from
Jan 17, 2017

Conversation

rococodogs
Copy link
Member

@rococodogs rococodogs commented Jan 13, 2017

This replaces the SearchResults list view with a tabular one that allows toggling of work fields.

TO DO

  • close field-select by clicking outside component
  • add defaults + reset-to-defaults
  • reimplement Gallery view
  • remove old ResultsListItem code

@rococodogs rococodogs force-pushed the search-results-table branch from 9e93faa to 6c3368b Compare January 16, 2017 19:39
@rococodogs
Copy link
Member Author

rococodogs commented Jan 16, 2017

just a note: the React dependencies needed to be updated to resolve a weird test bug (to the best I could find, see react/pull/8097). so when testing this PR / after it gets merged, be sure to run npm install

@rococodogs rococodogs changed the title [WIP] Search results table Add search results table Jan 17, 2017
@rococodogs rococodogs removed the request for review from jrgriffiniii January 17, 2017 16:54
@jrgriffiniii jrgriffiniii merged commit bff1924 into master Jan 17, 2017
@jrgriffiniii jrgriffiniii deleted the search-results-table branch January 17, 2017 19:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants