The function exports.loadModules in screenshotsLoader.js can take a tabId, but I'm not passing one in in background.js in the loadScreenshotUI function.
This does not seem to be affecting anything, but it's worth fixing in the next iteration (just add the tabId parameter to the line in background.js loading it, like this: selectorLoader.loadModules(tabId).
The function
exports.loadModulesinscreenshotsLoader.jscan take atabId, but I'm not passing one in inbackground.jsin theloadScreenshotUIfunction.This does not seem to be affecting anything, but it's worth fixing in the next iteration (just add the
tabIdparameter to the line in background.js loading it, like this:selectorLoader.loadModules(tabId).