refactor: bring down eslint warnings to zero (#1714)

- install `@eslint-react/eslint-plugin` dependency
- configure `@eslint-react/eslint-plugin` in eslint config
- modernize `lint` command in `package.json`
- disable or fix various reported lint issues
- fix `div` being nested in `p` for settings
- replace deprecated `event.keyCode` with `event.key`
- update isEscKeyPress method and unit tests which used deprecated `event.keyCode`
- allow `eslint` v8 as peer dependency for `@eslint-react/eslint-plugin`

1 files changed, 1 insertions, 1 deletions

diff --git a/src/stores/AppStore.ts b/src/stores/AppStore.ts
index b0a925411..51058c8eb 100644
--- a/src/stores/AppStore.ts
+++ b/src/stores/AppStore.ts
@@ -571,7 +571,7 @@ export default class AppStore extends TypedStore {
     debug(`Removed download ${id}`);
     if (id === null) {
       const indexesToRemove: number[] = [];
-      this.downloads.map(item => {
+      this.downloads.forEach(item => {
         if (!item.state) return;
         if (item.state === 'completed' || item.state === 'cancelled') {
           indexesToRemove.push(this.downloads.indexOf(item));