Code Monkey home page Code Monkey logo

Comments (7)

nekorushi avatar nekorushi commented on July 17, 2024 1

That's a cool improvement. I am not sure if that should be a separate set of shortcuts or should it replace the current ones (although i feel that your idea would be utilised more often, so it could replace current ones).
Personally I won't be able to look into it myself for foreseeable time due to personal responsibilities, but definitely +1 from me.

from asbplayer.

killergerbah avatar killergerbah commented on July 17, 2024

In this scenario, would the settings toggle specify the initial blur state when subtitles load? And the shortcuts only control the blur state during playback?

from asbplayer.

artjomsR avatar artjomsR commented on July 17, 2024

Yes, exactly.

If track blur is off in the settings:
blur is permanently off for that track and mouse hover / blur shortcut doesn't change the subtitle appearance

If track blur is on in the settings:

  • shortcut removes blur for the respective track, but blur is reintroduced when subtitles change
  • mouse hover works as is, no changes (unblurs on hover, resumes blur when cursor leaves the sub track)

from asbplayer.

artjomsR avatar artjomsR commented on July 17, 2024

I am not sure if that should be a separate set of shortcuts or should it replace the current ones

I think it is fine to replace the existing set of shortcuts. Whoever will use the blur subtitles will be using them regularly as part of their workflow, and I'm not expecting them to use the shortcuts to change the type of the flow that they use

In other words, if they're using blur, I'm expecting them to continue using the blur and not use the shortcuts to periodically toggle that setting on and off

from asbplayer.

killergerbah avatar killergerbah commented on July 17, 2024

I see, makes sense. I agree this would be better.

from asbplayer.

artjomsR avatar artjomsR commented on July 17, 2024

Ok I'll take a look at this then

from asbplayer.

artjomsR avatar artjomsR commented on July 17, 2024

I've created the PR but it's slow to respond to keypresses and not sure what's the best way to tackle that. Feel free to update the PR! :)

from asbplayer.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.