Sign upLogin

betajs/betajs-media-components

View on GitHub
Star

Showing 10,517 of 10,517 total issues

Line should be indented 2 spaces, but was indented 4 spaces
Open

    .#{$css}-topmessage-background{
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

Name of variable controlbar_primarybutton_font_size should be written in all lowercase letters with hyphens instead of underscores
Open

        font-size: $controlbar_primarybutton_font_size;
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

Name of variable controlbar_button_padding should be written in all lowercase letters with hyphens instead of underscores
Open

        padding: $controlbar_button_padding;
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

Opening curly brace { should be preceded by one space
Open

    .#{$css}-topmessage-background{
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

Name of variable font_family should be written in all lowercase letters with hyphens instead of underscores
Open

        font-family: $font_family;
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

Properties should be ordered color, display, font-family, font-size, font-weight, margin, padding, position
Open

        padding: $controlbar_button_padding;
Severity: Minor
Found in src/themes/video_recorder/theatre/topmessage.scss by scss-lint

TODO found
Open

                                // TODO: in the future could be better replace Async.eventually calling in the adsplaying change event to MutationObserver
Severity: Minor
Found in src/dynamics/video_player/sidebar/sidebar.js by fixme

TODO found
Open

                         "queryselector": null // TODO: will be shown on hover only on this element
Severity: Minor
Found in src/dynamics/video_player/player/player.js by fixme

TODO found
Open

                                visible: true, // TODO add parameter for setting source quality settings visibility
Severity: Minor
Found in src/dynamics/video_player/player/player.js by fixme

TODO found
Open

                    startUpdateVolume: function(args, element) { // TODO this was copied from the video player, should refactor to keep it DRY
Severity: Minor
Found in src/dynamics/ads_player/ads_controlbar/ads_controlbar.js by fixme

TODO found
Open

                    // TODO: add option for selection
Severity: Minor
Found in src/dynamics/ads_player/ads_player.js by fixme

TODO found
Open

                                        // TODO: need to be improved, this part runs only if first video from the playlist has error
Severity: Minor
Found in src/dynamics/video_player/sidebar/sidebar.js by fixme

TODO found
Open

                        this._videoElement = this.parent() && this.parent().activeElement().querySelector("[data-video='video']"); // TODO video element for outstream
Severity: Minor
Found in src/dynamics/ads_player/ads_player.js by fixme

TODO found
Open

      min-height: 100%; // TODO: check cube, theatre and default themes capability
Severity: Minor
Found in src/themes/_common/player/fullscreen.scss by fixme

TODO found
Open

                        this._canvasHeight = 34; // TODO calculate instead of hard coding value
Severity: Minor
Found in src/dynamics/video_recorder/trimmer/trimmer.js by fixme

TODO found
Open

                         "showonhover": false, // TODO: will be shown on hover only
Severity: Minor
Found in src/dynamics/video_player/player/player.js by fixme

TODO found
Open

                            // "fluidsidebar": true, // TODO: not works for now, if false, 50% width will be applied on sidebar
Severity: Minor
Found in src/dynamics/video_player/player/player.js by fixme
Severity
Category
Status
Source
Language