fix: invert calendar and clock icons in dark mode#5727
fix: invert calendar and clock icons in dark mode#5727tanujbhaud wants to merge 1 commit intocanonical:mainfrom
Conversation
Calendar and clock icons in date/time inputs were difficult to see in dark mode. Applied filter: invert(1) to webkit-calendar-picker-indicator pseudo-elements when inside .is-dark containers. Fixes canonical#5696
|
tanujbhaud is not a collaborator of the repo |
|
Hi @tanujbhaud , thanks for the contribution. Have you considered other alternatives than filter:invert(1) for this problem ? |
|
Hi @advl, thanks for reviewing! I did consider alternatives, and here's my reasoning: Main alternatives I looked into:
Why I chose filter: invert(1):
|
Summary
Fixes #5696
Calendar and clock icons in
type="date"andtype="time"inputs were barely visible in dark mode due to their default black color. This PR appliesfilter: invert(1)to these icons when inside.is-darkcontainers, making them white and clearly visible against dark backgrounds.Changes
::-webkit-calendar-picker-indicatorpseudo-elementsfilter: nonein light mode (default behavior)filter: invert(1)in dark mode (inside.is-darkcontainers)scss/_base_forms.scssImplementation Details
This fix is based on the solution implemented in lxd-ui (PR #1620) and adapted to vanilla-framework's dark mode pattern using
.is-darkclass.Testing
Expected Behavior
Light Mode: Calendar/clock icons remain black (default browser behavior)
Dark Mode: Calendar/clock icons are inverted to white for visibility
Screenshots
See original issue screenshots: #5696