Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cassettator-markers): css customization #8

Merged
merged 1 commit into from
Dec 3, 2023

Conversation

amtins
Copy link
Owner

@amtins amtins commented Dec 3, 2023

Description

Facilitates customization via CSS. It is no longer necessary to target the component class in order to redefine the CSS variable, thus improving the developer experience.

In concrete terms, this means the following changes:

  • Before modification:
.cst-marker {
  --cst-marker-played-background-color: blue;
}
  • After modification:
.video-js {
  --cst-marker-played-background-color: blue;
}

Changes made

  • remove the assignment of a default value to variables
  • add a new variable for the default background color
  • add documentation for the new variable

Facilitates customization via CSS. It is no longer necessary to target the
component class in order to redefine the CSS variable, thus improving the
developer experience.

In concrete terms, this means the following changes:

- Before modification:
  ```css
    .cst-marker {
        --cst-marker-played-background-color: blue;
    }
  ```
- After modification:
  ```css
    .video-js {
        --cst-marker-played-background-color: blue;
    }
  ```

- remove the assignment of a default value to variables
- add a new variable for the default background color
- add documentation for the new variable
@amtins amtins merged commit 5c40056 into main Dec 3, 2023
@amtins amtins deleted the feat/css-customization branch December 3, 2023 18:10
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