souporserious / react-media-player

React audio and video player.

Home Page:https://souporserious.github.io/react-media-player/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Clarify withMediaProps decorator usage

ismay opened this issue · comments

commented

This might be obvious to some, but it might be nice to clarify that components using the withMediaProps decorator only work when they're descendants of the <Media> component here.

Just a line explaining that would already be enough I think.

Sorry about that! I can definitely see how that can be confusing. I updated the docs to hopefully mitigate the confusion around this. I'm hoping to have a proper docs site sometime this year to help even more with this. Please feel free to open a PR if you'd like to add more clarification. Thanks for filing an issue!

commented

Cool! 👍 Thanks for the quick fix and the awesome library!