Skip to content

feat(time-filter): redesign modal with segmented toggle, Reset button, and Advanced disclosure#81

Merged
techtimo merged 3 commits into
mainfrom
feat/time-filter-modal-redesign
May 6, 2026
Merged

feat(time-filter): redesign modal with segmented toggle, Reset button, and Advanced disclosure#81
techtimo merged 3 commits into
mainfrom
feat/time-filter-modal-redesign

Conversation

@techtimo
Copy link
Copy Markdown
Owner

@techtimo techtimo commented May 6, 2026

Summary

  • Replace RadioControl with ToggleGroupControl (Single Day / Date Range) for a cleaner mode toggle at the top of the modal
  • Remove the "No filter" radio from each endpoint section; add a per-section Reset link that appears when a selection is active
  • Hide the relative option on the "Show points to" endpoint behind an Advanced disclosure — it's a rare edge case ("exclude the last N hours") that was cluttering the default view
  • Add scoped CSS to reset input[type="radio"] size inside the modal, fixing WP admin stylesheet interference in the shortcode generator / dashboard context

Also includes:

  • fix(markers): inject popup nav when images fail to load (was previously only injected on successful load)
  • chore: Prettier formatting cleanup across several files

techtimo added 3 commits May 6, 2026 12:40
…options

- Replace RadioControl with ToggleGroupControl (Single Day / Date Range)
- Remove 'No filter' radio; add per-endpoint Reset button instead
- Hide relative option on 'to' endpoint behind Advanced disclosure
- Scope radio size reset CSS to fix WP admin stylesheet interference
@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

❌ Patch coverage is 20.00000% with 16 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/map-engine/MarkerManager.ts 0.00% 13 Missing ⚠️
src/spotmap-admin/tabs/FeedsTab.jsx 0.00% 2 Missing ⚠️
src/spotmap-admin/components/ProviderSelector.jsx 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@techtimo techtimo merged commit aadb42c into main May 6, 2026
4 of 5 checks passed
@techtimo techtimo deleted the feat/time-filter-modal-redesign branch May 6, 2026 10:50
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