Skip to content

Add missing CSS for overlay opacity utility classes#819

Open
jjroelofs wants to merge 1 commit into
8.xfrom
jur/8.x/818-add-overlay-opacity-utility-classes
Open

Add missing CSS for overlay opacity utility classes#819
jjroelofs wants to merge 1 commit into
8.xfrom
jur/8.x/818-add-overlay-opacity-utility-classes

Conversation

@jjroelofs

Copy link
Copy Markdown
Collaborator

Linked issues

Solution

Added CSS rules for the overlay opacity modifier utility classes (dxpr-theme-util-overlay-20/40/60/80) that were registered in dxpr_theme.info.yml but had no CSS implementation. Each class now overrides the ::after pseudo-element opacity and the .YTPOverlay opacity to the corresponding value (0.2, 0.4, 0.6, 0.8).

Changes are in both scss/helpers/helper-classes.scss (source) and css/helpers/helper-classes.css (compiled artifact).

Checklist

  • I have read the CONTRIBUTING.md document.
  • My commit messages follow the contributing standards and style of this project.
  • My code follows the coding standards and style of this project.
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Need to run update.php after code changes
  • Requires a change to end-user documentation.
  • Requires a change to developer documentation.
  • Requires a change to QA tests.
  • Requires a new QA test.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

The overlay opacity modifier classes (overlay-20/40/60/80) were
registered in dxpr_theme.info.yml and visible in the DXPR Builder
class picker, but had no CSS implementation. The default overlay
opacity remained at 0.9 regardless of which modifier was selected.

Fixes #818

Co-Authored-By: Claude Opus 4.6 <[email protected]>
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.

1 participant