Jump to content

Web Client: ASS Render Improvement


Recommended Posts

UnkNow404
Posted (edited)

The ASS subtitle web render engine SubtitlesOctopus only need to bind video element, it doesn't need create canvas manually. Manually creating a canvas can easily cause issues with misalignment with the subtitle video.

It can remove the canvas and canvasParent two options in 'modules/htmlvideoplayer/plugin.js' while create SubtitlesOctopus instance.

self.currentSubtitlesOctopus = new responses({
    video: videoElement,
    subUrl: subContent ? null : textTrackUrl,
    subContent: subContent,
    workerUrl: _approuter.default.baseUrl() + "/bower_components/javascriptsubtitlesoctopus/dist/subtitles-octopus-worker.js",
    legacyWorkerUrl: _approuter.default.baseUrl() + "/bower_components/javascriptsubtitlesoctopus/dist/subtitles-octopus-worker-legacy.js",
    fallbackFont: function (track) {
        var fonts = [];
        !function (track) {
            var langsNotNeedingFont, track = (track.Language || "").toLowerCase();
            return !track || !(langsNotNeedingFont = ["dut", "nld", "nl", "eng", "en", "en-us", "en-gb", "fin", "fi", "fre", "fra", "fr", "ger", "deu", "de", "heb", "he", "hun", "hu", "ita", "it", "nor", "no", "pol", "pl", "por", "pt", "pob", "pt-br", "rus", "ru", "spa", "es", "es-mx", "es-419", "swe", "sv"]).includes(track) && !langsNotNeedingFont.includes(track.split("-")[0])
        }(track) || fonts.push(_approuter.default.baseUrl() + "/modules/fonts/GoNotoKurrent.woff2");
        return fonts
    }(track)[0],
    onError: function () {
        _htmlmediahelper.default.onErrorInternal(self, "mediadecodeerror")
    },
    lossyRender: null != window.createImageBitmap,
    renderMode: null != window.createImageBitmap ? "lossy" : "wasm-blend",
    detectAlphaBug: !_browser.default.chromecast,
    canvas: canvas,			# Should remove this
    canvasParent: canvasParent		# Should remove this
})

 

Edited by UnkNow404
Posted

Hello UnkNow404,

** This is an auto reply **

Please wait for someone from staff support or our members to reply to you.

It's recommended to provide more info, as it explain in this thread:


Thank you.

Emby Team

Posted

Hi, Thanks. We'll take a look.

  • Like 1

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...