add new logo colour field (as optional)

guardian · Jan 18, 2024 · 646db8a · 646db8a
1 parent c6d8345
commit 646db8a
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/app/models/BannerDesign.scala b/app/models/BannerDesign.scala
@@ -72,6 +72,7 @@ case class BannerDesignBasicColours(
   bodyText: HexColour,
   headerText: HexColour,
   articleCountText: HexColour,
+  logo: Option[HexColour]
 )
 
 case class BannerDesignHighlightedTextColours(

diff --git a/public/src/components/channelManagement/bannerDesigns/BasicColoursEditor.tsx b/public/src/components/channelManagement/bannerDesigns/BasicColoursEditor.tsx
@@ -2,6 +2,7 @@ import React from 'react';
 import { BasicColours } from '../../../models/bannerDesign';
 import { ColourInput } from './ColourInput';
 import { makeStyles, Theme } from '@material-ui/core/styles';
+import { stringToHexColour } from '../../../utils/bannerDesigns';
 
 const useStyles = makeStyles(({ spacing }: Theme) => ({
   container: {
@@ -62,6 +63,14 @@ export const BasicColoursEditor: React.FC<Props> = ({
         onChange={colour => onChange({ ...basicColours, articleCountText: colour })}
         onValidationChange={onValidationChange}
       />
+      <ColourInput
+        colour={basicColours.logo ?? stringToHexColour('000000')}
+        name="colours.basic.logo"
+        label="Guardian Logo Colour"
+        isDisabled={isDisabled}
+        onChange={colour => onChange({ ...basicColours, logo: colour })}
+        onValidationChange={onValidationChange}
+      />
     </div>
   );
 };
diff --git a/public/src/models/bannerDesign.ts b/public/src/models/bannerDesign.ts
@@ -12,6 +12,7 @@ export interface BasicColours {
   bodyText: HexColour;
   headerText: HexColour;
   articleCountText: HexColour;
+  logo?: HexColour;
 }
 
 export interface HighlightedTextColours {