From 94c0da28ce33fda7ec79f777797c2ec9a67e5aab Mon Sep 17 00:00:00 2001
From: Anton Krytskyi <ant.krytskyi@gmail.com>
Date: Wed, 13 Aug 2025 16:45:07 +0300
Subject: [PATCH] add background color prop to Brand

---
 admin/brands/forms.py                         |  1 +
 admin/brands/views.py                         |  8 ++++++++
 api/brands/serializers.py                     |  1 +
 osf/migrations/0032_brand_background_color.py | 18 ++++++++++++++++++
 osf/models/brand.py                           |  1 +
 osf_tests/factories.py                        |  1 +
 6 files changed, 30 insertions(+)
 create mode 100644 osf/migrations/0032_brand_background_color.py

diff --git a/admin/brands/forms.py b/admin/brands/forms.py
index 942d4929338..05693c66a05 100644
--- a/admin/brands/forms.py
+++ b/admin/brands/forms.py
@@ -11,6 +11,7 @@ class Meta:
         widgets = {
             'primary_color': TextInput(attrs={'class': 'colorpicker'}),
             'secondary_color': TextInput(attrs={'class': 'colorpicker'}),
+            'background_color': TextInput(attrs={'class': 'colorpicker'}),
             'topnav_logo_image': TextInput(attrs={'placeholder': 'Logo should be max height of 40px', 'size': 200}),
             'hero_logo_image': TextInput(
                 attrs={'placeholder': 'Logo image should be max height of 100px', 'size': 200}
diff --git a/admin/brands/views.py b/admin/brands/views.py
index 01b449d3fe8..11473f86939 100644
--- a/admin/brands/views.py
+++ b/admin/brands/views.py
@@ -81,6 +81,7 @@ def post(self, request, *args, **kwargs):
         view = BrandChangeForm.as_view()
         primary_color = request.POST.get('primary_color')
         secondary_color = request.POST.get('secondary_color')
+        background_color = request.POST.get('background_color')
 
         if not is_a11y(primary_color):
             messages.warning(request, """The selected primary color is not a11y compliant.
@@ -88,6 +89,9 @@ def post(self, request, *args, **kwargs):
         if not is_a11y(secondary_color):
             messages.warning(request, """The selected secondary color is not a11y compliant.
                 For more information, visit https://color.a11y.com/""")
+        if background_color and not is_a11y(background_color):
+            messages.warning(request, """The selected background color is not a11y compliant.
+                For more information, visit https://color.a11y.com/""")
         return view(request, *args, **kwargs)
 
 
@@ -109,6 +113,7 @@ def get_context_data(self, *args, **kwargs):
     def post(self, request, *args, **kwargs):
         primary_color = request.POST.get('primary_color')
         secondary_color = request.POST.get('secondary_color')
+        background_color = request.POST.get('background_color')
 
         if not is_a11y(primary_color):
             messages.warning(request, """The selected primary color is not a11y compliant.
@@ -116,4 +121,7 @@ def post(self, request, *args, **kwargs):
         if not is_a11y(secondary_color):
             messages.warning(request, """The selected secondary color is not a11y compliant.
                 For more information, visit https://color.a11y.com/""")
+        if background_color and not is_a11y(background_color):
+            messages.warning(request, """The selected background color is not a11y compliant.
+                For more information, visit https://color.a11y.com/""")
         return super().post(request, *args, **kwargs)
diff --git a/api/brands/serializers.py b/api/brands/serializers.py
index 8d040e7a93d..485d515b098 100644
--- a/api/brands/serializers.py
+++ b/api/brands/serializers.py
@@ -15,6 +15,7 @@ class BrandSerializer(JSONAPISerializer):
 
     primary_color = ser.CharField(read_only=True, max_length=7)
     secondary_color = ser.CharField(read_only=True, max_length=7)
+    background_color = ser.CharField(read_only=True, allow_null=True, max_length=7)
 
     links = LinksField({
         'self': 'get_absolute_url',
diff --git a/osf/migrations/0032_brand_background_color.py b/osf/migrations/0032_brand_background_color.py
new file mode 100644
index 00000000000..9b465e81e4a
--- /dev/null
+++ b/osf/migrations/0032_brand_background_color.py
@@ -0,0 +1,18 @@
+# Generated by Django 4.2.15 on 2025-08-12 12:48
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('osf', '0031_abstractprovider_registration_word'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='brand',
+            name='background_color',
+            field=models.CharField(blank=True, max_length=7, null=True),
+        ),
+    ]
diff --git a/osf/models/brand.py b/osf/models/brand.py
index ae4650407aa..5a857f5f3b1 100644
--- a/osf/models/brand.py
+++ b/osf/models/brand.py
@@ -23,6 +23,7 @@ class Meta:
 
     primary_color = models.CharField(max_length=7)
     secondary_color = models.CharField(max_length=7)
+    background_color = models.CharField(max_length=7, blank=True, null=True)
 
     def __str__(self):
         return f'{self.name} ({self.id})'
diff --git a/osf_tests/factories.py b/osf_tests/factories.py
index 7ad8885e1ad..ea1ab95d437 100644
--- a/osf_tests/factories.py
+++ b/osf_tests/factories.py
@@ -1299,6 +1299,7 @@ class Meta:
 
     primary_color = factory.Faker('hex_color')
     secondary_color = factory.Faker('hex_color')
+    background_color = factory.Faker('hex_color')
 
 
 class SchemaResponseFactory(DjangoModelFactory):