Skip to content

chore(search): remove SearchResultMapComponent from SearchResultsPage [closes #362]#364

Merged
zigzagdev merged 1 commit into
chore/define-following-mapfrom
chore/remove-map-from-search-results
May 19, 2026
Merged

chore(search): remove SearchResultMapComponent from SearchResultsPage [closes #362]#364
zigzagdev merged 1 commit into
chore/define-following-mapfrom
chore/remove-map-from-search-results

Conversation

@zigzagdev
Copy link
Copy Markdown
Owner

Summary

  • Removes <SearchResultMapComponent> usage from SearchResultsPage
  • Removes the now-unused import

Why

Map's purpose was unclear to users (review feedback). The list of cards is the primary interface for the exam-study use case.

Test plan

  • Search results page renders without the map
  • No console errors or TypeScript errors

🤖 Generated with Claude Code

…closes #362]

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Owner Author

@zigzagdev zigzagdev left a comment

Choose a reason for hiding this comment

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

ok

@zigzagdev zigzagdev merged commit 66a731e into chore/define-following-map May 19, 2026
1 check passed
@zigzagdev zigzagdev deleted the chore/remove-map-from-search-results branch May 19, 2026 01:07
zigzagdev added a commit that referenced this pull request May 19, 2026
* chore(search): remove SearchResultMapComponent from SearchResultsPage [closes #362] (#364)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore(search): delete SearchResultMapComponent.tsx [closes #363] (#366)

* chore(search): remove SearchResultMapComponent from SearchResultsPage [closes #362]

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* chore(search): delete SearchResultMapComponent.tsx [closes #363]

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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