Closed chriswhong closed 6 years ago
1) yarn.lock is committed, we shouldn't commit both yarn.lock
and package-lock.json
2) I think both the map and the table view should be grayed out or something if there is no result being displayed... we should make an issue for this, but I'm fine leaving it as-is for the time being.
Both good explanations. Merging!
Implements
react-table
to create a paginated table view to complement the map view.Moves navbar into
App
, so view button clicks can be handledCreates a top-level state property
rows
, which is used by the newTable
component.Refactors
/tiles/initialize
endpoint to returnrows
instead offeatureCount
(featureCount is calculated in the client from rows.length)For geojson mode, rows is calculated from the FeatureCollection once it is received in the client.
Cleans up unused dependencies
Closes #62