fpdcc / ccfp-asset-dashboard

CCFP Asset Dashboard
0 stars 1 forks source link

bring in ui polish with latest code + add interactions between map and table #132

Closed smcalilly closed 2 years ago

smcalilly commented 2 years ago

Overview

This PR has a ton of UI changes for issue #125. There are a lot of CSS changes and UI element tweaks.

It also includes some React code changes for #120 and #96. The changes let a user select a row in the table which will open a popup for that row's geometry and allow the user to save that geometry. Similarly, a user can select a single geometry and save the geometry.

Demo

Screen Shot 2022-02-16 at 3 56 30 PM Screen Shot 2022-02-16 at 3 56 41 PM Screen Shot 2022-02-16 at 3 57 06 PM Screen Shot 2022-02-16 at 3 57 36 PM Screen Shot 2022-02-16 at 3 57 53 PM

Notes

At a high level, made these UI changes:

Testing Instructions

to test the map changes:

fgregg commented 2 years ago

@smcalilly could you redeploy?

smcalilly commented 2 years ago

@fgregg here you go: https://ccfp-asset-d-ui-polish-tmvktzk.herokuapp.com

fgregg commented 2 years ago

looking slick. lets change visually change selected assets that have not been "saved" and let's also change the "save asset" button to "add asset to phase" button

smcalilly commented 2 years ago

@fgregg made the requested changes. in the process, i found an edge case that i fixed and attempted to articulate in this code comment. i also changed the name of some variables to be more clear, because i found that parts of the code was hard to understand......despite being the writer of it......

here's a demo of how it looks/works: select-asset