Skip to content

Update container classes for free user cap alerts

Doug Stull requested to merge 370503-adjust-container-classes into master

What does this MR do and why?

fixes the container classes to work properly with fluid and fixed layouts

Screenshots or screen recordings

before fluid before fixed after fluid after fixed(no change)
Screen_Shot_2022-09-01_at_4.15.17_PM Screen_Shot_2022-09-01_at_4.15.46_PM Screen_Shot_2022-09-01_at_3.52.38_PM Screen_Shot_2022-09-01_at_3.52.04_PM

How to set up and validate locally

  1. Apply this diff
diff --git a/ee/app/components/namespaces/free_user_cap/alert_component.rb b/ee/app/components/namespaces/free_user_cap/alert_component.rb
index 699a22c5018..2c431dd07d4 100644
--- a/ee/app/components/namespaces/free_user_cap/alert_component.rb
+++ b/ee/app/components/namespaces/free_user_cap/alert_component.rb
@@ -20,6 +20,7 @@ def initialize(namespace:, user:, content_class:)
       attr_reader :namespace, :user, :content_class

       def render?
+        return true
         return false unless Shared.default_render?(user: user, namespace: namespace)
         return false if dismissed?
  1. Go to any group overview, group usage quota or group/project member page.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #370503

Edited by Doug Stull

Merge request reports

Loading