Closed amtins closed 7 months ago
St.:grey_question: |
Category | Percentage | Covered / Total |
---|---|---|---|
π’ | Statements | 98.39% | 489/497 |
π’ | Branches | 93.44% | 228/244 |
π’ | Functions | 100% | 136/136 |
π’ | Lines | 99.16% | 473/477 |
153 tests passing in 9 suites.
Report generated by π§ͺjest coverage report action from e29fcf62d6646ba61b375da9221f48ce74efa783
Looks good to me! I've added some documentation to the wiki regarding how to integrate this skin: https://github.com/SRGSSR/pillarbox-web/wiki/Getting-started#quick-guide
Would be great to move this to the README once we've tested it and it works fine.
After discussion, it was decided that a specific task related to documentation would be carried out.
Description
Resolves #165 by offering a default
CSS theme
for the player interface. This default theme includes thevideo player
as well as theaudio player bar
.The folder architecture and file naming of the CSS files strictly respect those provided by
video.js
.Finally, all CSS rules are encapsulated in the theme class named
pillarbox-js
.Changes made
components
breakpoint
sizes