Skip to content

FIX: default.css > Invalid rgba() arguments: modules/_tables.scss#7354

Merged
donker merged 1 commit into
dnnsoftware:developfrom
Timo-Breumelhof:default_css/1.2RGBA
Jul 3, 2026
Merged

FIX: default.css > Invalid rgba() arguments: modules/_tables.scss#7354
donker merged 1 commit into
dnnsoftware:developfrom
Timo-Breumelhof:default_css/1.2RGBA

Conversation

@Timo-Breumelhof

Copy link
Copy Markdown
Contributor

No issue as minor issue

Summary

Invalid rgba() arguments
File: modules/_tables.scss:70-74 (.dnnTableFilter)
background: rgba(
var(--dnn-color-foreground, 0),
var(--dnn-color-foreground, 0),
var(--dnn-color-foreground, 0),
0.04);
Elsewhere in the same file (lines 50-54) the correct pattern is used:
separate -r/-g/-b channel variables as the three rgba() numeric arguments.
Here the full color variable (--dnn-color-foreground, which resolves to
something like #444) is used as all three channel arguments instead of
--dnn-color-foreground-r/-g/-b. rgba() expects numbers 0-255 for the first
three arguments, not a color value, so this declaration is invalid and is
dropped by the browser.

Solution:
background: rgba(
var(--dnn-color-foreground-r, 0),
var(--dnn-color-foreground-g, 0),
var(--dnn-color-foreground-b, 0),
0.04);

@Timo-Breumelhof Timo-Breumelhof changed the title Invalid rgba() arguments in modules/_tables.scss FIX: Invalid rgba() arguments in modules/_tables.scss Jul 3, 2026
@Timo-Breumelhof Timo-Breumelhof changed the title FIX: Invalid rgba() arguments in modules/_tables.scss FIX: Default.css > Invalid rgba() arguments in modules/_tables.scss Jul 3, 2026
@Timo-Breumelhof Timo-Breumelhof changed the title FIX: Default.css > Invalid rgba() arguments in modules/_tables.scss FIX: Default.css > Invalid rgba() arguments: modules/_tables.scss Jul 3, 2026
@Timo-Breumelhof Timo-Breumelhof changed the title FIX: Default.css > Invalid rgba() arguments: modules/_tables.scss FIX: default.css > Invalid rgba() arguments: modules/_tables.scss Jul 3, 2026
@donker donker added this to the 10.3.3 milestone Jul 3, 2026
@donker
donker added this pull request to the merge queue Jul 3, 2026
Merged via the queue into dnnsoftware:develop with commit 5f85725 Jul 3, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants