django-cms / djangocms-admin-style

django CMS Admin Style is a Django Theme tailored to the needs of django CMS.

Home Page:http://www.django-cms.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Header box-shadow missing (CSS variables do not work in rgba)

petrklus opened this issue · comments

The box-shadow seems to be broken in 3.1.0:

image

When inspecting, seems that the colour is not being picked up:

image

While in 3.0.0, it is resolved into a colour:

image

I believe that this construct is not valid:

rgba(var(--dca-black), 0.2, which results from

$toolbar-shadow: 0 0 5px rgba($black, 0.2);

The solution is to either hard-code the colour value, or to define colours with alpha in them and then use them throughout.

I believe this would also affect $hover-tooltip-bgcolor, $dropdown-shadow and many others that now use CSS variables inside the rgba construct.

Tested on Mac OS 10.15.7, Chrome 100.0.4896.60

@fsbraun I belive the above is related to the dark mode introduction, which changed the simple black/white colours (which worked in rgba construct) to CSS variables, which do not seem to resolve.

Would you be up for a new PR/amending yours to fix the above?

@petrklus I will look into it. Anyways, the shadows should have stayed black. Otherwise you'll get white shadows in dark mode which is strange. Will also check if other shadows might be affected. Thanks for pointing this out!

Thank you @fsbraun ! Will keep an eye on the PR and test once released.