stemroller icon indicating copy to clipboard operation
stemroller copied to clipboard

Feature: Select local files

Open alectrocute opened this issue 3 years ago • 1 comments

Proposed changes –

  • Allow selecting local files via quick n' easy click.

Notes –

  • For a rainy day: I feel like we could refactor a lot of the processQueue code to make it less tied into YouTube and more source-agnostic.

alectrocute avatar Aug 09 '22 00:08 alectrocute

This is a great start! Really like how you put that "or select a local file" link right along with the default description. I think you're right - processQueue does need to be refactored, and that's part of the reason I'd been waiting to implement this for a while. Thank you for the PR - may take a look at it in a few days (once the Mac compatibility bugs are fixed) and see if there might be a way to do a little refactoring/cleaning before finalizing this feature. Looking forward to having this implemented!

iffyloop avatar Aug 09 '22 02:08 iffyloop

Went ahead and added this feature so we can close the PR for now if that's OK with you. I did things a little differently (see this commit for details) but your work here gave me some inspiration to get started. We can continue the discussion in #3, and maybe make a new issue in the future for refactoring.

iffyloop avatar Aug 12 '22 05:08 iffyloop