Skip to content
Snippets Groups Projects

Detail navigation

Merged Manuel Jung requested to merge detail-navigation into main

Since wo do not want a full blown up navigation library like react-router i implemented a own navigation solution which can just navigate by updating the search param for the detail page

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Resolved by Adam Reichold

      Since wo do not want a full blown up navigation library like react-router

      I also think this is a bit of misrepresentation: I do not want to carve out a routing hole for the SPA's router in the HTTP server configuration of the CMS. I am certainly open to using a full blown SPA router here, we just need to limit the routing to happen based on the query string instead of the path.

  • Manuel Jung added 1 commit

    added 1 commit

    • 4c4fae92 - prefer zustan over react context

    Compare with previous version

  • Manuel Jung added 1 commit

    added 1 commit

    • acb7e6ff - respect all query params on "isDetailPage" selector

    Compare with previous version

  • Manuel Jung added 1 commit

    added 1 commit

    Compare with previous version

  • Manuel Jung marked this merge request as ready

    marked this merge request as ready

  • Manuel Jung added 2 commits

    added 2 commits

    • bf9405f4 - renamed prop Link.query to Link.to
    • dee4f40f - rely on useHitId instead of manual query-string checking

    Compare with previous version

  • Manuel Jung added 1 commit

    added 1 commit

    • 0b55b876 - Merge branch 'main' into detail-navigation

    Compare with previous version

  • Manuel Jung added 1 commit

    added 1 commit

    • eda87f67 - rely on window.URL instead of raw href string

    Compare with previous version

  • Manuel Jung added 1 commit

    added 1 commit

    • 4ab88202 - made useNavigation hook aware of other search params

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading

    Consent

    On this website, we use the web analytics service Matomo to analyze and review the use of our website. Through the collected statistics, we can improve our offerings and make them more appealing for you. Here, you can decide whether to allow us to process your data and set corresponding cookies for these purposes, in addition to technically necessary cookies. Further information on data protection—especially regarding "cookies" and "Matomo"—can be found in our privacy policy. You can withdraw your consent at any time.