Skip to content

Map tab optimizations#601

Merged
higs4281 merged 66 commits intomainfrom
rad-geo-updates
Apr 16, 2026
Merged

Map tab optimizations#601
higs4281 merged 66 commits intomainfrom
rad-geo-updates

Conversation

@flacoman91
Copy link
Copy Markdown
Contributor

@flacoman91 flacoman91 commented Apr 9, 2026 •
edited
Loading

Additions

  • Added external GeoLegend from CE

Removals

  • Remove per 1,000 functionality from map tab
  • removed internal legend in tilemap

Changes

  • Updated dev harness to cap the width max to 1230 px with 30 L/R gutters
  • Updated tile map styling
  • Updated tile map popup tooltip
  • Update tile map focus hover states etc
  • CSS cleanup, removed body-copy class
  • Converted p tag to use DSR paragraph

Testing

Screenshots

Notes

Todos

Checklist

  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows the standards laid out in the front end playbook
  • Passes all existing automated tests
  • New functions include new tests
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged
  • Visually tested in supported browsers and devices
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 9, 2026 •
edited
Loading

Coverage Status

Coverage is 88.168% — rad-geo-updates into main. No base build found for main.

@flacoman91
remove in map legend
c17cc0a 
update geo legend and add to project

update code

tooltip caret

caret

cleaned up tooltip

tooltip cleanup

cleanup, convert to some more DSR things

linting

update dist

cleanup code

update dist

adjust buttons

save dist

udate jest thresh

fixing bug, remove complaint per warning

restore border to tiles

update css
@wpears wpears marked this pull request as ready for review April 16, 2026 21:18
wpears
wpears approved these changes Apr 16, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@wpears wpears left a comment

Choose a reason for hiding this comment

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

Code looks good.

@higs4281 higs4281 added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit 0f7efa8 Apr 16, 2026
1 check passed
@higs4281 higs4281 deleted the rad-geo-updates branch April 16, 2026 21:44
@flacoman91 flacoman91 mentioned this pull request Apr 16, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wpears wpears wpears approved these changes

@higs4281 higs4281 Awaiting requested review from higs4281

@anselmbradford anselmbradford Awaiting requested review from anselmbradford

@contolini contolini Awaiting requested review from contolini

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@flacoman91 @coveralls @wpears @higs4281