Skip to content

feat: add locale toggle and Japanese description on heritage detail#294

Merged
zigzagdev merged 3 commits intochore/implement-description_ja-frontendfrom
feat/add-new-property-container_viewcomponent
Apr 22, 2026
Merged

feat: add locale toggle and Japanese description on heritage detail#294
zigzagdev merged 3 commits intochore/implement-description_ja-frontendfrom
feat/add-new-property-container_viewcomponent

Conversation

@zigzagdev
Copy link
Copy Markdown
Owner

  • Wire locale toggle button in HeritageDetailLayout to update lang query param
  • Render heritageNameJp and shortDescriptionJp when locale is ja
  • Navigate to results page with search params on sub-header submit
  • Update tests for new shortDescriptionJp property

Wire toggleLocale callback from container to layout, render a flag button in
the tab bar, and switch HeritageHero between Japanese and English titles
based on locale.
Branch on locale in the overview section to show shortDescriptionJp when ja
locale is active, falling back to the English shortDescription otherwise.
Plumb locale through HeritageDetailLayout and update fixtures accordingly.
Build a URLSearchParams from the current search values and push to
/heritages/results when the sub-header form is submitted so submissions
from the detail page reach the results view instead of staying in place.
@zigzagdev zigzagdev force-pushed the feat/add-new-property-container_viewcomponent branch from 6ac561e to 6695fec Compare April 22, 2026 09:33
@zigzagdev zigzagdev linked an issue Apr 22, 2026 that may be closed by this pull request
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.

LGTM

@zigzagdev zigzagdev merged commit 312fdd5 into chore/implement-description_ja-frontend Apr 22, 2026
1 check passed
@zigzagdev zigzagdev deleted the feat/add-new-property-container_viewcomponent branch April 22, 2026 11:34
@zigzagdev zigzagdev self-assigned this Apr 22, 2026
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.

Render the field in the Presentational component

1 participant