Skip to content

Conversation

@therobrob
Copy link
Member

@therobrob therobrob commented Nov 3, 2025

  1. removed arrow-icon from print view (was shown in booking-expanders)
  2. refactored open-status with pseudo-selector ::details-content
  3. some general code refactoring

@netlify
Copy link

netlify bot commented Nov 3, 2025

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit 3e76ede
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/691a381bbb159700084ece07
😎 Deploy Preview https://deploy-preview-455--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

github-actions bot commented Nov 3, 2025

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

@therobrob therobrob added the technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc. label Nov 3, 2025
Copy link
Member

@MoritzWeber0 MoritzWeber0 left a comment

Choose a reason for hiding this comment

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

Really cool! 🥳

@MoritzWeber0 MoritzWeber0 changed the title refactor: expander print view refactor: Simplify expander print view styling Nov 16, 2025
@therobrob
Copy link
Member Author

now the icons are centered on multi-line-headings either 🙈 But we can also leave it as it is.
IMG_9612

@MoritzWeber0
Copy link
Member

now the icons are centered on multi-line-headings either 🙈 But we can also leave it as it is.

But it's consistent with how we do it for the train categories too. Let's leave it for this PR and if there is a need, we can open it as additional PR or track it as issue.

@MoritzWeber0 MoritzWeber0 merged commit bbfa547 into main Nov 16, 2025
7 checks passed
@MoritzWeber0 MoritzWeber0 deleted the refactor/expander-print-view branch November 16, 2025 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants