Skip to content

Commit

Permalink
Fix eslint errors after bump to 6.6.0 (#1595)
Browse files Browse the repository at this point in the history
* Bump eslint from 3.19.0 to 6.6.0

Bumps [eslint](https://github.com/eslint/eslint) from 3.19.0 to 6.6.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v3.19.0...v6.6.0)

Signed-off-by: dependabot[bot] <[email protected]>

* Fix (and suppress) eslint errors
  • Loading branch information
stoically authored Dec 18, 2019
1 parent 3c2dda5 commit 426e81b
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 13 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
"addons-linter": "^1.3.2",
"ajv": "^6.6.2",
"chai": "^4.1.2",
"eslint": "^3.17.1",
"eslint": "^6.6.0",
"eslint-plugin-no-unsanitized": "^2.0.0",
"eslint-plugin-promise": "^3.4.0",
"htmllint-cli": "0.0.7",
Expand Down
1 change: 1 addition & 0 deletions src/js/background/messageHandler.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,7 @@ const messageHandler = {
if (!extensionInfo.permissions.includes("contextualIdentities")) {
throw new Error("Missing contextualIdentities permission");
}
// eslint-disable-next-line require-atomic-updates
externalExtensionAllowed[sender.id] = true;
}
let response;
Expand Down
25 changes: 13 additions & 12 deletions src/js/popup.js
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,7 @@ const Logic = {

getPanelSelector(panel) {
if (this._onboardingVariation === "securityOnboarding" &&
// eslint-disable-next-line no-prototype-builtins
panel.hasOwnProperty("securityPanelSelector")) {
return panel.securityPanelSelector;
} else {
Expand Down Expand Up @@ -570,21 +571,21 @@ Logic.registerPanel(P_CONTAINERS_LIST, {
break;
}
case 39:
{
const showTabs = element.parentNode.querySelector(".show-tabs");
if(showTabs) {
showTabs.click();
}
break;
{
const showTabs = element.parentNode.querySelector(".show-tabs");
if(showTabs) {
showTabs.click();
}
break;
}
case 37:
{
const hideTabs = document.querySelector(".panel-back-arrow");
if(hideTabs) {
hideTabs.click();
}
break;
{
const hideTabs = document.querySelector(".panel-back-arrow");
if(hideTabs) {
hideTabs.click();
}
break;
}
default:
if ((e.keyCode >= 49 && e.keyCode <= 57) &&
Logic._currentPanel === "containersList") {
Expand Down

0 comments on commit 426e81b

Please sign in to comment.