Project

General

Profile

Actions

Bug #39593

open

Apply pf-v5-c-page--BackgroundColor to #rails-app-content

Added by Samir Jha 3 days ago. Updated 3 days ago.

Status:
Ready For Testing
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
easy
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Claude summary:

Title: Rails-rendered pages missing background color after stylelint scoping change

Description:
Commit fb3d37d ("Fixes #39276 - Add Stylelint rule to block global css overrides") changed the selector in layout.scss from .pf-v5-c-page to
#foreman-page to comply with the new foreman/no-root-pf-overrides stylelint rule. This inadvertently dropped the --pf-v5-c-page--BackgroundColor
override for #rails-app-content, which is the wrapper for Rails-rendered pages in base.html.erb and uses class="pf-v5-c-page" instead of
id="foreman-page".
As a result, Rails-rendered pages (used by Foreman and plugins like Katello) no longer inherit the intended background color
(--pf-v5-global--BackgroundColor--100).
Fix: Add --pf-v5-c-page--BackgroundColor: var(--pf-v5-global--BackgroundColor--100) to the existing #rails-app-content.pf-v5-c-page block in
app/assets/stylesheets/base.scss.
Actions #1

Updated by The Foreman Bot 3 days ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/11134 added
Actions

Also available in: Atom PDF