Skip to content

random ui/ux fixes#196

Merged
shaynesidman merged 11 commits intomainfrom
tika/ui-fixes
Apr 13, 2026
Merged

random ui/ux fixes#196
shaynesidman merged 11 commits intomainfrom
tika/ui-fixes

Conversation

@tika
Copy link
Copy Markdown
Collaborator

@tika tika commented Apr 13, 2026

  • fix: consistent header spacing on pages
  • feat: responsive chart page
  • feat: add responsive actions to map page too
  • fix: export issue when switching tabs
  • fix: make whole column clickable
  • fix: click event being swallowed on schools page sort
  • feat: responsive map page
  • fix: if no map date, do not display any year

closes #155, #90

Favour filter popover over single toggle
CleanShot 2026-04-13 at 01 27 36

@tika tika marked this pull request as ready for review April 13, 2026 05:26
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
mhd Ready Ready Preview, Comment Apr 13, 2026 4:04pm

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 13, 2026

PR Summary

Medium Risk
Mostly UI/UX refactors, but it changes map layer rendering/visibility and export behavior (chart ref handling), which could impact user-visible functionality across key pages.

Overview
Improves responsiveness on the chart and map pages by moving filters and share/actions into mobile-friendly Sheet/Popover UIs, while keeping full sidebars/actions on larger breakpoints.

Fixes chart export reliability when switching between bar/line by wrapping the animated chart area in a stable container ref, and updates cart/empty-state iconography (ShoppingBasket).

Adds new heatmap toggles (showHeatmap, showRegions) persisted in the URL and wires them through useHeatmapLayers to control MapLibre layer visibility; also tweaks heatmap title formatting to omit the year when not selected.

Makes table sorting/resizing interactions less error-prone by making sortable column headers fully clickable and preventing resize handles from triggering sort clicks.

Reviewed by Cursor Bugbot for commit 96538ad. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 96538ad. Configure here.

Co-authored-by: tika <tika@users.noreply.github.com>
@shaynesidman shaynesidman merged commit 472bbd3 into main Apr 13, 2026
1 of 2 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.

Design: Hide Schools Toggle

3 participants