Calendar Headers color issue

tgcowell / waves

This is a blend of 2 themes found within the Home Assistant community. Inspired mostly by Noctis, I've adjust colours slightly and have also opted to pull some features from Caule Theme packs to build my own 'ultimate' theme. I will continue to update overtime and do my best to credit those whom I have 'referenced'

82 stars 12 forks source link

Calendar Headers color issue #6

Closed slodziak closed 3 years ago

slodziak commented 3 years ago

Really nice theme, appreciate the hard work! However, it seems that the dark mode version suffers when the HA native calendar card is present. The header part shows in light color instead of following the rest of the theme.

calendar

Thank you!

tgcowell commented 3 years ago

Hi @slodziak,

Thanks for the heads up, will try and work out what it's using and provide an update shortly.

Thanks 🙏

tgcowell commented 3 years ago

Hi @slodziak,

I've been slowly chipping away at this issue and here is what I have found so far.

If you remove the background the bars will then reflect what the actual css or yaml file. Which is var(--ha-card-background) so in theory it should all blend in as you have clearly suggested.
I've got to do some further testing with other themes that also include backgrounds, to date the ones I've based this off also have the same fault. So I'm curious is this a fault within that theme or it's set to happen and I can't resolve it.

I'm still working away so I will keep you in the loop, sorry I couldn't get this one resolve quickly.

tgcowell commented 3 years ago

I have found the issue... just not sure why it's happening. The var(--ha-card-background) is rgba set at .15 opacity so they are see through. If I make it 1 so it's not clear the issue is resolve. So for some reason it doesn't like that. At least it's a step in the right direction.

tgcowell commented 3 years ago

Hi @slodziak

I've made a minor change that helps remove the contrasting 'white' table row and moves it into a more dull grey. It's got something to do with the actual -fc-neutral-bg-color but despite several attempts to change this the core theme would always override my variables.

While it's not what I was hoping to do, it's the best I can provide for now. I'm going to close it as I believe it's an improvement, however please note I'll see what I can do moving forward.

Thanks again

slodziak commented 3 years ago

Appreciate the work! It's definitely an improvement!