From d98ac9530483d189f8ccdcafde31e2da6596cf4b Mon Sep 17 00:00:00 2001 From: Mylanos Date: Mon, 26 May 2025 17:27:45 +0200 Subject: [PATCH] OCPBUGS-56379: Using PF defaults for masthead logo max width CSS styling fixes the problem, where certain logos were not displaying at all. The custom logo images rendered within the Brand component scale well even without these overrides, the static logos appearance within ReactSVG component remained the same as well. --- frontend/public/style/_overrides.scss | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/frontend/public/style/_overrides.scss b/frontend/public/style/_overrides.scss index 7402e1973f7..73534c2f2d9 100644 --- a/frontend/public/style/_overrides.scss +++ b/frontend/public/style/_overrides.scss @@ -2,8 +2,6 @@ // Use this file to override styles from 3rd party dependencies -$masthead-logo-max-height: 60px; - // TODO: Remove when upgrading to PF 6.3.0: https://github.com/patternfly/patternfly/pull/7420 .catalog-tile-pf-title { @include co-break-word; @@ -178,15 +176,6 @@ $masthead-logo-max-height: 60px; height: 100%; } -.pf-v6-c-masthead { - --pf-v6-c-masthead__logo--MaxHeight: $masthead-logo-max-height; // so that logos with three lines of text fit -} - -.pf-v6-c-masthead__logo { - --pf-v6-c-masthead__logo--MaxHeight: #{$masthead-logo-max-height}; // Restore the max-height from PageHeader to maintain backwards compatibility - --pf-v6-c-masthead__logo--Width: auto; // Do not set a width to maintain backwards compatibility -} - // PF components that calculate their correct height based on --pf-t--global--font--size--md: 1rem .pf-v6-c-modal-box, .pf-v6-c-switch {