Add video component

[ischaojie]

fixes #54

[codecov]

Codecov Report

Merging #71 (e40da5c) into main (c9e8bb9) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff            @@
##              main       #71   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines          118       118           
=========================================
  Hits           118       118           

[codecov]

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

[samuelcolvin]

Looking good, thanks so much.

please rebase, resolve conflicts and make sure tests are passing - I suspect you'll need to do a bit of work to make sure the JSON Schemas match.

You also need to add video to the list of components in the demo index page.

By the way, please can you use fixes #54 in the PR body so the issue is automatically closed when the PR is merged, otherwise I'll forget. see here.

[ischaojie]

What's your opinion of the comments here #54 (comment) ?

[samuelcolvin]

I think this looks great as a start, let's merge this once fixed and see what people think.

[samuelcolvin]

I think this looks great as a start, let's merge this once fixed and see what people think.

[samuelcolvin]

thanks so much.