Skip to content

Updated brand colours.#152

Open
AlexSkrypnyk wants to merge 2 commits intodevelopfrom
feature/update-colours
Open

Updated brand colours.#152
AlexSkrypnyk wants to merge 2 commits intodevelopfrom
feature/update-colours

Conversation

@AlexSkrypnyk
Copy link
Member

@AlexSkrypnyk AlexSkrypnyk commented Mar 13, 2026

Summary by CodeRabbit

  • Style
    • Updated brand color palette for both light and dark themes — refreshed primary, secondary, and tertiary brand colors (six values changed) to improve visual consistency and contrast across the application.

@coderabbitai
Copy link

coderabbitai bot commented Mar 13, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: d5f20d42-ec45-4db4-9ff0-61b6d0c84d41

📥 Commits

Reviewing files that changed from the base of the PR and between 398920c and 93f5967.

⛔ Files ignored due to path filters (1)
  • composer.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • composer.json

📝 Walkthrough

Walkthrough

Updates two areas: (1) brand color hex values in the design system Sass map for both light and dark themes; (2) dependency version constraint bumps in composer.json. No behavioral or control-flow changes.

Changes

Cohort / File(s) Summary
Theme Brand Colors
web/themes/custom/drevops/components/variables.base.scss
Modified the $ct-colors-brands Sass map: updated hex values for brand1, brand2, and brand3 in both light and dark branches (6 color replacements).
Composer dependencies
composer.json
Bumped multiple dependency version constraints (require and require-dev). No code logic changes; only manifest/version updates.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'Updated brand colours' accurately describes the primary change in the changeset: updating color values in the brand color palette within the SCSS variables file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/update-colours
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@web/themes/custom/drevops/components/variables.base.scss`:
- Around line 17-24: Update the PR by running WCAG contrast and visual
regression checks for the theme color tokens defined in variables.base.scss —
specifically the 'light' and 'dark' maps and their 'brand1', 'brand2', 'brand3'
entries — across impacted UI surfaces (text on brand backgrounds, buttons,
links). Use an automated contrast checker (e.g., axe-core, pa11y, or Lighthouse)
and a visual regression tool (e.g., Percy, Chromatic, or Playwright snapshots)
to validate both themes; if any checks fail, adjust the token values in the
'light'/'dark' maps or update component styles to meet AA/AAA thresholds and
attach failing screenshots/diffs and updated token changes to the PR for
reviewer verification.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 106146a2-7dcd-48fb-900e-19049fabd64d

📥 Commits

Reviewing files that changed from the base of the PR and between 7463c54 and 398920c.

📒 Files selected for processing (1)
  • web/themes/custom/drevops/components/variables.base.scss

Comment on lines +17 to +24
'brand1': #d4725e,
'brand2': #f8f4f2,
'brand3': #1a2836,
),
'dark': (
'brand1': #61daff,
'brand2': #003a4f,
'brand3': #00698f,
'brand1': #e8a090,
'brand2': #1a2836,
'brand3': #d4725e,
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick | 🔵 Trivial

Run contrast/visual regression checks for both themes before merge.

Given these are core brand tokens, please validate impacted UI surfaces (e.g., text on brand backgrounds, buttons, links) for WCAG contrast in light and dark themes.

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@web/themes/custom/drevops/components/variables.base.scss` around lines 17 -
24, Update the PR by running WCAG contrast and visual regression checks for the
theme color tokens defined in variables.base.scss — specifically the 'light' and
'dark' maps and their 'brand1', 'brand2', 'brand3' entries — across impacted UI
surfaces (text on brand backgrounds, buttons, links). Use an automated contrast
checker (e.g., axe-core, pa11y, or Lighthouse) and a visual regression tool
(e.g., Percy, Chromatic, or Playwright snapshots) to validate both themes; if
any checks fail, adjust the token values in the 'light'/'dark' maps or update
component styles to meet AA/AAA thresholds and attach failing screenshots/diffs
and updated token changes to the PR for reviewer verification.

```
hangelogs summary:

 - drupal/core-composer-scaffold updated from 11.3.3 to 11.3.5 patch
   See changes: drupal/core-composer-scaffold@11.3.3...11.3.5
   Release notes: https://github.com/drupal/core-composer-scaffold/releases/tag/11.3.5

 - symfony/string updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/string@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/string/releases/tag/v7.4.6

 - symfony/console updated from v7.4.4 to v7.4.7 patch
   See changes: symfony/console@v7.4.4...v7.4.7
   Release notes: https://github.com/symfony/console/releases/tag/v7.4.7

 - symfony/finder updated from v7.4.5 to v7.4.6 patch
   See changes: symfony/finder@v7.4.5...v7.4.6
   Release notes: https://github.com/symfony/finder/releases/tag/v7.4.6

 - symfony/filesystem updated from v7.4.0 to v7.4.6 patch
   See changes: symfony/filesystem@v7.4.0...v7.4.6
   Release notes: https://github.com/symfony/filesystem/releases/tag/v7.4.6

 - symfony/yaml updated from v7.4.1 to v7.4.6 patch
   See changes: symfony/yaml@v7.4.1...v7.4.6
   Release notes: https://github.com/symfony/yaml/releases/tag/v7.4.6

 - symfony/translation updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/translation@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/translation/releases/tag/v7.4.6

 - symfony/dependency-injection updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/dependency-injection@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/dependency-injection/releases/tag/v7.4.7

 - symfony/config updated from v7.4.4 to v7.4.7 patch
   See changes: symfony/config@v7.4.4...v7.4.7
   Release notes: https://github.com/symfony/config/releases/tag/v7.4.7

 - symfony/http-client updated from v6.4.33 to v6.4.34 patch
   See changes: symfony/http-client@v6.4.33...v6.4.34
   Release notes: https://github.com/symfony/http-client/releases/tag/v6.4.34

 - symfony/css-selector updated from v7.4.0 to v7.4.6 patch
   See changes: symfony/css-selector@v7.4.0...v7.4.6
   Release notes: https://github.com/symfony/css-selector/releases/tag/v7.4.6

 - symfony/dom-crawler updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/dom-crawler@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/dom-crawler/releases/tag/v7.4.6

 - symfony/validator updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/validator@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/validator/releases/tag/v7.4.7

 - symfony/serializer updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/serializer@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/serializer/releases/tag/v7.4.7

 - symfony/routing updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/routing@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/routing/releases/tag/v7.4.6

 - symfony/http-foundation updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/http-foundation@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/http-foundation/releases/tag/v7.4.7

 - symfony/mime updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/mime@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/mime/releases/tag/v7.4.7

 - symfony/mailer updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/mailer@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/mailer/releases/tag/v7.4.6

 - symfony/var-dumper updated from v7.4.4 to v7.4.6 patch
   See changes: symfony/var-dumper@v7.4.4...v7.4.6
   Release notes: https://github.com/symfony/var-dumper/releases/tag/v7.4.6

 - symfony/http-kernel updated from v7.4.5 to v7.4.7 patch
   See changes: symfony/http-kernel@v7.4.5...v7.4.7
   Release notes: https://github.com/symfony/http-kernel/releases/tag/v7.4.7

 - guzzlehttp/psr7 updated from 2.8.0 to 2.8.1 patch
   See changes: guzzle/psr7@2.8.0...2.8.1
   Release notes: https://github.com/guzzle/psr7/releases/tag/2.8.1

 - drupal/core updated from 11.3.3 to 11.3.5 patch
   See changes: drupal/core@11.3.3...11.3.5
   Release notes: https://github.com/drupal/core/releases/tag/11.3.5

 - drupal/drupal-driver updated from v2.3.0 to v2.4.0 minor
   See changes: jhedstrom/DrupalDriver@v2.3.0...v2.4.0
   Release notes: https://github.com/jhedstrom/DrupalDriver/releases/tag/v2.4.0

 - drupal/drupal-extension updated from v5.1.0 to v5.2.1 minor
   See changes: jhedstrom/drupalextension@v5.1.0...v5.2.1
   Release notes: https://github.com/jhedstrom/drupalextension/releases/tag/v5.2.1

 - drevops/behat-steps updated from 3.5.0 to 3.5.2 patch
   See changes: drevops/behat-steps@3.5.0...3.5.2
   Release notes: https://github.com/drevops/behat-steps/releases/tag/3.5.2

 - drupal/ai updated from 1.2.9 to 1.3.0 minor

 - drupal/ai_provider_openai updated from 1.2.0 to 1.2.1 patch

 - drupal/webform updated from 6.3.0-beta7 to 6.3.0-beta8 patch

 - drupal/linkit updated from 7.0.12 to 7.0.13 patch

 - drupal/crop updated from 2.5.0 to 2.6.0 minor

 - slevomat/coding-standard updated from 8.27.1 to 8.28.0 minor
   See changes: slevomat/coding-standard@8.27.1...8.28.0
   Release notes: https://github.com/slevomat/coding-standard/releases/tag/8.28.0

 - drupal/core-recommended updated from 11.3.3 to 11.3.5 patch
   See changes: drupal/core-recommended@11.3.3...11.3.5
   Release notes: https://github.com/drupal/core-recommended/releases/tag/11.3.5

 - drupal/scheduled_transitions updated from 2.8.3 to 2.8.4 patch

 - psy/psysh updated from v0.12.20 to v0.12.21 patch
   See changes: bobthecow/psysh@v0.12.20...v0.12.21
   Release notes: https://github.com/bobthecow/psysh/releases/tag/v0.12.21

 - laravel/prompts updated from v0.3.13 to v0.3.14 patch
   See changes: laravel/prompts@v0.3.13...v0.3.14
   Release notes: https://github.com/laravel/prompts/releases/tag/v0.3.14

 - phpstan/phpstan updated from 2.1.39 to 2.1.40 patch

 - mglaman/phpstan-drupal updated from 2.0.10 to 2.0.11 patch
   See changes: mglaman/phpstan-drupal@2.0.10...2.0.11
   Release notes: https://github.com/mglaman/phpstan-drupal/releases/tag/2.0.11

 - rector/rector updated from 2.3.7 to 2.3.8 patch
   See changes: rectorphp/rector@2.3.7...2.3.8
   Release notes: https://github.com/rectorphp/rector/releases/tag/2.3.8

 - webmozart/assert updated from 2.1.5 to 2.1.6 patch
   See changes: webmozarts/assert@2.1.5...2.1.6
   Release notes: https://github.com/webmozarts/assert/releases/tag/2.1.6

 - phpdocumentor/type-resolver updated from 1.12.0 to 2.0.0 major
   See changes: phpDocumentor/TypeResolver@1.12.0...2.0.0
   Release notes: https://github.com/phpDocumentor/TypeResolver/releases/tag/2.0.0

 - phpdocumentor/reflection-docblock updated from 5.6.6 to 6.0.2 major
   See changes: phpDocumentor/ReflectionDocBlock@5.6.6...6.0.2
   Release notes: https://github.com/phpDocumentor/ReflectionDocBlock/releases/tag/6.0.2

 - phpspec/prophecy updated from v1.25.0 to v1.26.0 minor
   See changes: phpspec/prophecy@v1.25.0...v1.26.0
   Release notes: https://github.com/phpspec/prophecy/releases/tag/v1.26.0

 - vincentlanglet/twig-cs-fixer updated from 3.13.0 to 3.14.0 minor
   See changes: VincentLanglet/Twig-CS-Fixer@3.13.0...3.14.0
   Release notes: https://github.com/VincentLanglet/Twig-CS-Fixer/releases/tag/3.14.0

No security vulnerability advisories found.
```
@github-actions
Copy link

Code Coverage Report:
  2026-03-13 23:53:56

 Summary:
  Classes: 50.00% (5/10)
  Methods: 73.91% (17/23)
  Lines:   92.77% (308/332)

Drupal\do_feed\FeedUrlBuilder
  Methods: 100.00% ( 4/ 4)   Lines: 100.00% ( 18/ 18)
Drupal\do_feed\Form\FeedSettingsForm
  Methods: 100.00% ( 4/ 4)   Lines: 100.00% ( 15/ 15)
Drupal\do_feed\Hook\EntityDeleteHook
  Methods:  50.00% ( 1/ 2)   Lines:  92.31% ( 12/ 13)
Drupal\do_feed\Hook\EntityPresaveHook
  Methods: 100.00% ( 4/ 4)   Lines: 100.00% ( 54/ 54)
Drupal\do_feed\Hook\PreprocessParagraphHook
  Methods: 100.00% ( 2/ 2)   Lines: 100.00% ( 14/ 14)
Drupal\do_feed\Hook\PreprocessViewsViewRowRssHook
  Methods: 100.00% ( 1/ 1)   Lines: 100.00% (  3/  3)
Drupal\do_feed\Hook\ViewsPreViewHook
  Methods:  50.00% ( 1/ 2)   Lines:  96.43% ( 27/ 28)

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.73%. Comparing base (7463c54) to head (93f5967).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #152   +/-   ##
========================================
  Coverage    87.73%   87.73%           
========================================
  Files           11       11           
  Lines          163      163           
========================================
  Hits           143      143           
  Misses          20       20           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions bot added the CONFLICT Pull request has a conflict that needs to be resolved before it can be merged label Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CONFLICT Pull request has a conflict that needs to be resolved before it can be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants