Skip to content

bugfix: Firefox mobile navbar (dropdown parents without a URL couldn't be opened) - #92

Merged
dgnad merged 1 commit into
icaps2026:mainfrom
medjakub:main
Jul 6, 2026
Merged

bugfix: Firefox mobile navbar (dropdown parents without a URL couldn't be opened)#92
dgnad merged 1 commit into
icaps2026:mainfrom
medjakub:main

Conversation

@medjakub

@medjakub medjakub commented Jul 3, 2026

Copy link
Copy Markdown

On mobile Firefox, tapping a dropdown parent that has no page of its own (Students, Attending) did nothing useful. Those items were rendered as with the expand arrow as a nested inside the anchor, so tapping the arrow both toggled the submenu checkbox and fired the empty-href navigation — and Firefox resolved that in favor of navigating, so the submenu never opened.

On top of that, the arrow-rotation rule (input:checked + .drop-icon) never actually matched, since the drop-icon isn't an adjacent sibling of the checkbox in our markup.

Changes:

  • Dropdown parents with no URL now render as a (toggle only, no navigation); real links stay as .
  • Switched the arrow-rotation selector to .menu__dropdown:has(> input:checked) so it matches the actual structure.
  • Moved the mobile hover/active background from .menu__item to .menu__link so only the tapped row highlights.

@dgnad
dgnad merged commit 5769617 into icaps2026:main Jul 6, 2026
@dgnad

dgnad commented Jul 6, 2026

Copy link
Copy Markdown

Works like a charm. Many thanks!

@medjakub

medjakub commented Jul 6, 2026

Copy link
Copy Markdown
Author

FYI, I've found a nice solution by Kevin (@kzhe0012) in the repository of ICAPS 2024 webpage. It is currently under review to be merged to ICAPS 2027 webpage. We may switch to this fix as well, but given the conference is over, it might not be worth the time (since the previous fix is already in).

@dgnad

dgnad commented Jul 6, 2026

Copy link
Copy Markdown

I've also made the 2027 chairs aware that merging the change would be nice. Either solution is fine, I guess. For the 2026 page there is probably no need to make it "nicer" :)

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.

2 participants