Skip to content

[IMP] web_company_color: Compile SCSS#3427

Open
SirPyTech wants to merge 1 commit intoOCA:18.0from
PyTech-SRL:18.0-imp-web_compny_color-compile_scss
Open

[IMP] web_company_color: Compile SCSS#3427
SirPyTech wants to merge 1 commit intoOCA:18.0from
PyTech-SRL:18.0-imp-web_compny_color-compile_scss

Conversation

@SirPyTech
Copy link

@SirPyTech SirPyTech commented Feb 2, 2026

Otherwise function calls are not evaluated in the created attachment.

Right now it is not needed because the created attachment does not use any functions, but if any were added, that would not be evaluated.

For instance, try editing the created CSS by adding a call to the desaturate function like:

.app-menu-container {
    background: url('/web_responsive/static/src/img/home-menu-bg-overlay.svg'),
        linear-gradient(
            to bottom,
            %(color_navbar_bg)s,
            desaturate(lighten(%(color_navbar_bg)s, 20%%), 15)
        );
}

that would be sent to the client as-is resulting in something like:

.app-menu-container {
    background: url('/web_responsive/static/src/img/home-menu-bg-overlay.svg'),
        linear-gradient(
            to bottom,
            #123456,
            desaturate(lighten(#123456, 20%), 15)
        );
}

that is not acceptable CSS, and thus is not evaluated.

Other SCSS assets are automatically compiled in their bundle, but this one skips that procedure because of the overrides for the web_company_color.company_color_assets bundle.

@SirPyTech SirPyTech marked this pull request as draft February 2, 2026 11:37
Otherwise function calls are not evaluated in the created attachment
@SirPyTech SirPyTech force-pushed the 18.0-imp-web_compny_color-compile_scss branch from 1705b71 to 177d482 Compare February 2, 2026 12:02
@SirPyTech SirPyTech marked this pull request as ready for review February 2, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant