IONOS(theme): fix darkmode for logo #100
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the dark mode styling for the IONOS logo by consolidating the CSS rules. The media query for dark mode color scheme is moved from a standalone block into a nested rule within the .logo selector, improving code organization and maintainability.
- Nested the dark mode media query directly within the
.logoselector - Removed the duplicate standalone media query block targeting
#header .logo
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @media (prefers-color-scheme: dark) { | ||
| filter: invert(100%) saturate(1%) brightness(99%) contrast(200%); | ||
| } | ||
| } |
There was a problem hiding this comment.
The indentation is inconsistent. The media query and its contents use two tabs, but should use one tab to align with the parent .logo selector's nesting level.
| @media (prefers-color-scheme: dark) { | |
| filter: invert(100%) saturate(1%) brightness(99%) contrast(200%); | |
| } | |
| } | |
| @media (prefers-color-scheme: dark) { | |
| filter: invert(100%) saturate(1%) brightness(99%) contrast(200%); | |
| } | |
| } |
printminion-co
force-pushed
the
ms/fix/darkmode_logo
branch
from
c485633 to
596fa1e
Compare
|
this last 596fa1e commit breaks logo color on login page: it's orange in both light and dark mode. |
Will check... |
Additional increase logo size on login Signed-off-by: Matthias Sauer <[email protected]>
- same size as on non-public page - supports darkmode now - add darkmode for subtitle (e.g. shared folder name) Signed-off-by: Matthias Sauer <[email protected]>
matsaur
force-pushed
the
ms/fix/darkmode_logo
branch
from
596fa1e to
78c6dd9
Compare
Signed-off-by: Tatjana Kaschperko Lindt <[email protected]>
this avoids overlapping with logo Signed-off-by: Tatjana Kaschperko Lindt <[email protected]>
moved media query for logo darkmode to proper location