Skip to content

fix: don't use transparent sidebars on mobile#3861

Merged
mnkiefer merged 4 commits into
mainfrom
fix-hamburger-menu
Nov 13, 2025
Merged

fix: don't use transparent sidebars on mobile#3861
mnkiefer merged 4 commits into
mainfrom
fix-hamburger-menu

Conversation

@mnkiefer
Copy link
Copy Markdown
Collaborator

@mnkiefer mnkiefer commented Nov 13, 2025

  • Use solid backgrounds on mobile for better for readability. Keeping it transparent (as on Desktop) makes it illegible.
image

@mnkiefer mnkiefer marked this pull request as ready for review November 13, 2025 12:21
Copilot AI review requested due to automatic review settings November 13, 2025 12:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves mobile readability by ensuring solid backgrounds are used on mobile devices instead of transparent ones, which were causing legibility issues. The transparent backgrounds are now only applied on desktop viewports (≥769px).

Key Changes

  • Wrapped all transparent background CSS rules (for both light and dark themes) in @media (min-width: 769px) queries to restrict them to desktop only
  • Modified sidebar transparency rules to apply only on desktop, ensuring mobile sidebars have solid backgrounds
  • Added comprehensive mobile navigation styles to improve header visibility and usability on smaller screens

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/src/styles/custom.css Outdated
Comment thread docs/src/styles/custom.css Outdated
mnkiefer and others added 2 commits November 13, 2025 13:25
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mnkiefer mnkiefer merged commit a38b521 into main Nov 13, 2025
12 checks passed
@mnkiefer mnkiefer deleted the fix-hamburger-menu branch November 13, 2025 12:27
@mnkiefer mnkiefer self-assigned this Nov 24, 2025
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