Skip to content

Conversation

@subodhr258
Copy link
Collaborator

@subodhr258 subodhr258 commented Aug 11, 2025

This pull request makes a small visual update to the admin interface by changing the accent color used for WordPress components in the #root-easydam and #root-video-editor sections. The accent color is now set to #ab3a6c instead of using the $godam-primary-text-color variable.

Copilot AI review requested due to automatic review settings August 11, 2025 16:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes the accent color used in the Video Editor component by replacing a SCSS variable reference with a hardcoded hex color value.

  • Replaces the SCSS variable variables.$godam-primary-text-color with the hardcoded color #ab3a6c for the --wp-components-color-accent CSS custom property

#root-easydam,
#root-video-editor {
--wp-components-color-accent: variables.$godam-primary-text-color;
--wp-components-color-accent: #ab3a6c;
Copy link

Copilot AI Aug 11, 2025

Choose a reason for hiding this comment

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

Using a hardcoded color value instead of the existing SCSS variable reduces maintainability. Consider defining this color as a SCSS variable (e.g., $video-editor-accent-color: #ab3a6c;) to maintain consistency with the existing color system and make future color changes easier.

Copilot uses AI. Check for mistakes.
@subodhr258 subodhr258 merged commit e5c0d31 into main Aug 11, 2025
2 checks passed
@subodhr258 subodhr258 deleted the hotfix/video-editor-ui branch August 11, 2025 16:10
@subodhr258
Copy link
Collaborator Author

Fixed: #872
Partially Fixed: #884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants