Skip to content

bcwat-815-819-sidebar-population-enhancement#833

Merged
idaeusice merged 25 commits intodevfrom
bcwat-815-819-sidebar-population-enhancement
Mar 18, 2026
Merged

bcwat-815-819-sidebar-population-enhancement#833
idaeusice merged 25 commits intodevfrom
bcwat-815-819-sidebar-population-enhancement

Conversation

@idaeusice
Copy link
Contributor

Description

  • Updated the current method of getting in-view map points to use javascript filtering over the mapbox queryRenderedFeatures method, which has shown to be slower.
  • Included various fixes for filtering mechanics. These appeared to be unworking previously, and were not getting set correctly based on the viewed
  • Added a couple small fixes around the MapFilters component and simplified some of the processes around getting and setting filters in general.

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@MaxDouglas1991
Copy link
Contributor

image The foundry icon is missing in the attribution

@MaxDouglas1991
Copy link
Contributor

image Are there filters based on the properties found in the list of points? I'm pretty sure current is supposed to be in these filters

@MaxDouglas1991
Copy link
Contributor

LGTM. There's some things to address but they are unrelated to this PR

@idaeusice idaeusice merged commit 1739a19 into dev Mar 18, 2026
5 of 6 checks passed
@idaeusice idaeusice deleted the bcwat-815-819-sidebar-population-enhancement branch March 18, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend vue, js, html, styling, or other client libraries (BCWAT-96)

Projects

None yet

3 participants