audio-only-youtube
audio-only-youtube copied to clipboard
Refactoring background.js
I refactored background.js to make the extension support enabling/disabling per tab. This closes issue #3
@sheldonled Thanks but I couldn't get it to working. Is it working fine in your local environment ?
Hey @Ashish-Bansal, I think I might have found the issue. Sent another commit to this PR, please take a look at it.
cheers
@sheldonled First of all, Thanks a lot for code cleanup! I really appreciate that. When I created this extension, I was a C++ guy ;)
I works as expected but I don't want to remove the previous behaviour too. Instead I would like to have enable/disable per tab as optional feature which gets (de)/activated through context menus ?
And sorry for late reply!
@Ashish-Bansal We should close it now.
@Kompwu There was one feature in this PR which still needs to be ported. That's why I haven't closed it.
@Ashish-Bansal individual toggle per tab?