Skip to content

Conversation

@MoritzWeber0
Copy link
Member

@MoritzWeber0 MoritzWeber0 commented Oct 22, 2025

Update the z-indices according to the following order (lowest to highest):

.o-interactive-map__controls (removed z-index, had no effect)
.o-dropdown__menu (removed z-index, should inherit from context)
.overlay--content
.o-aside
.overlay--search
#search (+ .pagefind-ui__form::before)
.o-header
.o-header__skip-link
.o-header__curtain
.o-header__nav--open nav
.a-snackbar

I took the line number and multiplied it by 2 to give some flexibility to add new indices in between in the future.

Resolves #432
Resolves #425
Resolves #406

@netlify
Copy link

netlify bot commented Oct 22, 2025

Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit 14d3609
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/68fa46ba0aab0500089f2596
😎 Deploy Preview https://deploy-preview-434--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

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 marked this pull request as draft October 22, 2025 11:18
@MoritzWeber0 MoritzWeber0 marked this pull request as ready for review October 22, 2025 11:57
@MoritzWeber0 MoritzWeber0 force-pushed the fix/z-index branch 2 times, most recently from 09c590f to c400fc7 Compare October 22, 2025 12:20
Copy link
Member

@therobrob therobrob left a comment

Choose a reason for hiding this comment

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

LGTM

@therobrob therobrob merged commit fbbf0f5 into main Oct 23, 2025
7 checks passed
@therobrob therobrob deleted the fix/z-index branch October 23, 2025 15:26
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.

Overlay doesn't blur the search button Language selection overlays mobile content menu Focus indicator cut off in mobile menu

3 participants