havit / Havit.Blazor

Free Bootstrap 5 components for ASP.NET Blazor + optional enterprise-level stack for Blazor development (gRPC code-first, layered architecture, localization, auth, ...)
https://havit.blazor.eu
MIT License
475 stars 66 forks source link

[HxInputDate] dark mode broken in Edge #750

Closed resignationinfinite closed 1 month ago

resignationinfinite commented 7 months ago

Only happens in Edge so far as I can tell.

Video: https://github.com/havit/Havit.Blazor/assets/10275357/3640c98f-ec38-42e2-bade-13f485a34190

hakenr commented 6 months ago

@crdo I'm able to reproduce the reported issue. The issue disappears when I force :hover on the select element: image

hakenr commented 6 months ago

2024-02-13_10-24-36

crdo commented 4 months ago

@hakenr to me this feels more like a browser bug rather than issue in our library.

hrocha1 commented 1 month ago

Right now I'm able to replicate this issue on all major browsers on Windows (Chrome, Edge, Firefox). On Firefox it's not working properly even in the light color theme.

I did some digging and it looks like the issue is with this line https://github.com/havit/Havit.Blazor/blob/ac2480613d0241df976d1e17b5911333f5a9065f/Havit.Blazor.Components.Web.Bootstrap/Calendar/HxCalendar.razor.css#L39 in .form-select:not(:hover) CSS selector.

If I turn it off in my dev tools this particular issue seems to go away, but I guess that the line is there for a reason and it might break something else if it's removed. Still probably a good start while trying to figure out this bug.

crdo commented 1 month ago

Although I still believe this is a browser bug and I cannot reproduce it as it works on all browsers across macOS I rewrote the CSS to get rid of that silly :not(:hover) selector. @hakenr please let me know if it helped or make a pre-release so @hrocha1 or @resignationinfinite can test it.

hrocha1 commented 1 month ago

Might be an issue with some Windows native UI elements. I haven't been able to reproduce it on macOS or Linux either.

hakenr commented 1 month ago

release 4.6.11-pre2, make sure you clear your browser cache (previous version of CSS bundle)

crdo commented 1 month ago

After a quick testing with @alexax578 I think it is fixed.

hrocha1 commented 1 month ago

@crdo Can confirm it works on my side, thanks 👍

crdo commented 1 month ago

Closing as fixed. Will be part of the 4.6.11 release.