Skip to content
Snippets Groups Projects

Global maps layout

Merged Hayley Corson-Dosch requested to merge global_maps_layout into main
1 unresolved thread

Description

This MR updates the global threats map page to add the top threat maps and spots in the layout for additional placeholder text.

Changes Made

  • Add new section at top of page for top threat maps
    • Set up w/ toggle so that individual threat categories can be added/removed from map
    • Adjust mapping code to not plot basin where a given threat is not the top threat and add a base map with all polygons (to set up maps to work with toggle)
  • Update placeholders for sub-threat text
    • Place all text in collapsible accordions
    • Show drivers and potential impacts text by default
  • Add the findex logo to the project page

image

image

image

How to Test

Pull changes locally and run npm run dev to preview in browser.

MR t-shirt size

  • extra-small (e.g., update color scheme)
  • small (e.g., add tooltip to chart)
  • medium (e.g., add new static chart, add new pipeline output)
  • large (e.g., initial data processing pipeline, interactive chart)

Timeline for review

  • ASAP - blocker for deployment
  • Today - blocker for ongoing work
  • Next couple of days
  • This week
  • When reviewer has availability

Review Needs

Does this MR include data processing, modeling, or visualization code that will require domain review prior to release?

  • Yes, and I have opened an issue to document the need for review, using the DomainReview issue template
  • Yes, and a domain review issue already exists
  • No

Related Issues

NA

Additional Notes

Only used placeholder text for now - will need to update later.

Merge Request Checklists

  • Code changes adhere to best practices documented in README.md
  • Code has been cleaned the way Vue likes it - run npm run lint --fix
  • If applicable, the need for future domain review has been documented in an issue
  • Below section documents which browsers the site has been tested on:
    • Desktop/laptop
      • Chrome
      • Safari
      • Edge
      • Firefox
    • Mobile device
      • Chrome
      • Safari
      • Edge
      • Firefox

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading