Skip to content

Conversation

@pujitakalinadhabhotla
Copy link

Screenshot 2025-03-26 at 10 26 20 PM

@mahekagg
Copy link

Lgtm! Could you make the default rendering of the map legend to have 'Available Sites' selected? Also, initialize all of the feature types to be as selected by default.

@pujitakalinadhabhotla
Copy link
Author

Screenshot 2025-04-03 at 11 00 06 AM

@hams7504
Copy link

hams7504 commented Apr 7, 2025

Looks and works great! Thank you.
One small nitpick: could you adjust the design so that the caret fits and is centered within the blue box at the bottom.
It looks like part of it is outside the box from what I see:

image

@pujitakalinadhabhotla
Copy link
Author

Screenshot 2025-04-08 at 8 01 29 PM

@hams7504
Copy link

hams7504 commented Apr 9, 2025

Screenshot 2025-04-08 at 8 01 29 PM

Looks a lot better, thank you! Could you make it more centered vertically, if possible? It seems like it's positioned too much to the bottom. Sorry for being so nitpicky! Just want this product to look clean for our client.

@mahekagg
Copy link

mahekagg commented Apr 9, 2025

Can you change the colors of the icons that populate on the map page to match the color on the legend sidebar? Currently, the adopted/inactive sites that display are red and not yellow/gray.

@mahekagg
Copy link

mahekagg commented Apr 9, 2025

Also, could you please make sure that the 'Other' icon shows up when 'Other' is selected on the map legend. Thank you!

@pujitakalinadhabhotla
Copy link
Author

updated!
Screenshot 2025-04-10 at 1 24 18 AM

Screenshot 2025-04-10 at 1 24 38 AM Screenshot 2025-04-10 at 1 24 55 AM

Copy link

@hams7504 hams7504 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for your changes!

@mahekagg mahekagg merged commit 717c6a3 into main Apr 15, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants