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

Add primaryColor prop 🎨 #41

Merged
merged 1 commit into from
Oct 28, 2021
Merged

Add primaryColor prop 🎨 #41

merged 1 commit into from
Oct 28, 2021

Conversation

third774
Copy link
Contributor

Removing tsdx lint since it's currently broken, and adding a clean
script to clear files that prettier shouldn't be concerned with.

Copy link

@Everlag Everlag left a comment

Choose a reason for hiding this comment

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

One question, otherwise LGTM.

src/types.ts Outdated
@@ -64,6 +64,10 @@ export interface StreamPlayerApi {
* The <video> element does not force the browser to follow the value of this attribute; it is a mere hint. Even though the preload="none" option is a valid HTML5 attribute, Stream player will always load some metadata to initialize the player. The amount of data loaded in this case is negligable.
*/
preload: Preload;
/**
* Any valid CSS color provided will be applied to certain elements of the player's UI.
Copy link

Choose a reason for hiding this comment

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

May be useful to include some examples; folks reading this may assume this limits to named css colors rather than being able to use rgba/hsl/etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

How about Any valid CSS color value (based on this MDN page: https://developer.mozilla.org/en-US/docs/Web/CSS/color_value)

Copy link

Choose a reason for hiding this comment

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

Sounds good to me 👍

Removing tsdx lint since it's currently broken, and adding a clean
script to clear files that prettier shouldn't be concerned with.
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.

2 participants