Skip to content

feat: Improve expander handling in search#888

Merged
MoritzWeber0 merged 3 commits intofix/search-hidden-booking-sectionsfrom
fix/headings-map
Apr 27, 2026
Merged

feat: Improve expander handling in search#888
MoritzWeber0 merged 3 commits intofix/search-hidden-booking-sectionsfrom
fix/headings-map

Conversation

@MoritzWeber0
Copy link
Copy Markdown
Member

@MoritzWeber0 MoritzWeber0 commented Apr 24, 2026

Titles of booking expanders and train categories do define a hidden heading. This has a few advantages:

  • The heading map is no longer broken, can be checked via the HeadingsMap extension.
  • Content of the next expander doesn't count as element from the previous expander.
  • Train categories and booking platforms title matches are ranked higher in the search
  • If the result targets a booking or train category expander, the page automatically scrolls to it.

Similar to the already existing behaviour with train category expanders, it's now possible to link to booking platforms directly, e. g. https://deploy-preview-888--fipguide.netlify.app/de/operator/oebb/#oebb-website works now.

Resolves #887
Resolves #462

TODO:

  • Handle accessibility

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 24, 2026

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit a687946
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/69ee7d4a1f27b800083c98c3
😎 Deploy Preview https://deploy-preview-888--fipguide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown

Hi there! 👋

Thank you for your contribution to the FIP Guide! 🚀
We appreciate your effort in making rail staff travel information more accessible. 🚄

Checklist before merging:

  • Added a description to the Pull Request
  • Checked the License of new pictures (non-commercial use without attribution)
  • Modified content in English
  • Modified content in German
  • Modified content in French

@MoritzWeber0 MoritzWeber0 changed the base branch from main to fix/search-hidden-booking-sections April 26, 2026 20:43
@MoritzWeber0 MoritzWeber0 merged commit a687946 into fix/search-hidden-booking-sections Apr 27, 2026
5 checks passed
@MoritzWeber0 MoritzWeber0 deleted the fix/headings-map branch April 27, 2026 08:39
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.

Handling of expanders in search Check headings map on operator pages

1 participant