-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FE] (feat): Inventory panel: Protected Areas [MRXN23-239] #1456
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
onClick={onSortChange} | ||
/> | ||
</th> | ||
{columns['name'] && ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This approach will not scale. Can we pass an array of headers to the table an just render them? Like:
const headers = [
{
text: 'Name',
name: 'name',
columns,
sorting,
onSortChange,
...,
{
...
}
]
Correct me if I am wrong, but when I add several PAs to the map, we should probably iterate over different colours so we can differentiate them. Also, the name of the PA should be listed in the legend below |
@andresgnlez As mentioned, the same color is maintained for the protected preview areas (blue) but the legend should show the list of WDPAs on the map |
36d7fda
to
d0919e5
Compare
Protected Areas by project
NOTE: on hold until API is ready.
Overview
This PR includes the inventory panel protected areas section with MOCKED DATA
Actions included:
Designs
https://www.figma.com/file/0xD4gR9pCqU7NROnlSW6ck/Marxan-Layout---Redesign?node-id=605%3A28284&mode=dev
Testing
Feature relevant tickets
MRXN23-239