Skip to content
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

Toggle a BVAP layer #278

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Toggle a BVAP layer #278

wants to merge 2 commits into from

Conversation

mapmeld
Copy link
Member

@mapmeld mapmeld commented Feb 6, 2025

Draft/demo of BVAP toggle on Colorado maps, trying to show a proof-of-concept so we can figure out a good UI

Screenshot 2025-02-05 at 10 48 02 PM Screenshot 2025-01-28 at 10 25 25 PM

Feedback

  • can we update the population layer upload process so that populations are uploaded as integers and not strings? I don't know how it affects storage or overhead, and it is manageable in the map styling code, just wanted to mention it
  • I think that this data might be showing % Black, and not the % BVAP

Questions:

  • should the layer be in Evaluation, in its own sidebar accordion section, or added to the map toolbar?
  • should the map key be over the map or in the same section as the toolbar?
  • currently hardcoding which maps have population data. Should this / is this on the DB record?

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.

1 participant