Skip to content

Remove redundant Catalog tab from store page#328

Merged
zortos293 merged 1 commit intodevfrom
capy/remove-redundant-catalog-tab
Apr 16, 2026
Merged

Remove redundant Catalog tab from store page#328
zortos293 merged 1 commit intodevfrom
capy/remove-redundant-catalog-tab

Conversation

@zortos293
Copy link
Copy Markdown
Collaborator

@zortos293 zortos293 commented Apr 16, 2026

This PR removes the redundant Catalog toggle from the store page and cleans up unused props/state plumbing, since HomePage is always rendered in catalog mode on the home route.

Main changes:

  • HomePage (opennow-stable/src/renderer/src/components/HomePage.tsx):

    • Removed source and onSourceChange props from HomePageProps
    • Removed the toolbar's home-tabs div containing the dead Catalog button
  • App (opennow-stable/src/renderer/src/App.tsx):

    • Removed GameSource type and source state variable
    • Simplified loadGames handler to use inline literal union type
    • Updated home-page refresh effect to key off currentPage !== "home" instead of source !== "main"
    • Removed stale setSource calls from catalog browse and reset handlers
    • Removed source and onSourceChange props from HomePage call site
  • Styles (opennow-stable/src/renderer/src/styles.css):

    • Removed unused .home-tabs and .home-tab selectors including hover/active/disabled variants

The rest of the toolbar (search, filters, sort, counts, and game grid) remains unchanged.

Open OPE-072 OPE-072

@zortos293 zortos293 added the capy Generated by capy.ai label Apr 16, 2026 — with Capy AI
@zortos293 zortos293 merged commit 72e1898 into dev Apr 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

capy Generated by capy.ai

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant