Skip to content

Conversation

@MoritzWeber0
Copy link
Member

They were not centered before:

image

@github-actions
Copy link

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

@netlify
Copy link

netlify bot commented Jan 20, 2026

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit 2338023
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/69714a002b85df000738ba1b
😎 Deploy Preview https://deploy-preview-626--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.

@therobrob
Copy link
Member

We can do so, but the icons will be centered either, when the text wraps.

Bildschirmfoto 2026-01-21 um 09 53 15 Bildschirmfoto 2026-01-21 um 09 54 34 Bildschirmfoto 2026-01-21 um 09 55 08

@MoritzWeber0
Copy link
Member Author

We can do so, but the icons will be centered either, when the text wraps.

Bildschirmfoto 2026-01-21 um 09 53 15 Bildschirmfoto 2026-01-21 um 09 54 34 Bildschirmfoto 2026-01-21 um 09 55 08

Fair, I'll try to find another solution where the text is centered, but the icons not.

@MoritzWeber0
Copy link
Member Author

MoritzWeber0 commented Jan 21, 2026

I changed if for the icons, but I'm not sure about the operators logos.

Which of the two options do you prefer? Left or right? @therobrob

image image

@therobrob
Copy link
Member

therobrob commented Jan 21, 2026

I changed if for the icons, but I'm not sure about the operators logos.

Which of the two options do you prefer? Left or right? @therobrob

Thank you! The operator-labels aren’t that long as the news-labels, so i prefer the right one. It looks better because text is centered vertically. This corresponds to the code of the PR, so it can be merged :)

@therobrob therobrob merged commit bc3bf4f into main Jan 21, 2026
7 checks passed
@therobrob therobrob deleted the fix/center-aside-links branch January 21, 2026 21:51
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.

3 participants