@internetarchive/bookreader 5.0.0-82 → 5.0.0-84

package/.eslintrc.js

@@ -5,35 +5,35 @@ module.exports = {
     "jquery": true,
     "es6": true,
     "node": true,
-    "jest": true
+    "jest": true,
   },
   "plugins": [
     "testcafe",
-    "no-jquery"
+    "no-jquery",
   ],
   "extends": [
     "eslint:recommended",
     "plugin:testcafe/recommended",
-    "plugin:no-jquery/deprecated"
+    "plugin:no-jquery/deprecated",
   ],
   "globals": {
     // Browser API
-    "MediaMetadata": "readonly"
+    "MediaMetadata": "readonly",
   },
   "parserOptions": {
     "ecmaVersion": 2020,
     "ecmaFeatures": { "legacyDecorators": true },
-    "sourceType": "module"
+    "sourceType": "module",
   },
   "rules": {
-    "comma-dangle": ["error", "only-multiline"],
+    "comma-dangle": ["error", "always-multiline", {"functions": "only-multiline"}],
     "semi": ["error", "always"],
     "space-before-blocks": ["error"],
     "keyword-spacing": [2, {"before": true, "after": true}],
     "indent": ["error", 2, {
       // This for some reason causing errors after switch to parser: babel-eslint.
       // See https://github.com/babel/babel-eslint/issues/681
-      "ignoredNodes": ["TemplateLiteral"]
+      "ignoredNodes": ["TemplateLiteral"],
     }],
     "no-console": "off", // Used too often behind `debug` options; not dealing with this now
     "no-empty": ["error", { "allowEmptyCatch": true }],
@@ -46,6 +46,6 @@ module.exports = {
     "no-var": ["error"],
     "prefer-const": ["error"],
     "space-infix-ops": ["error", { "int32Hint": false }],
-    "eol-last": ["error", "always"]
-  }
+    "eol-last": ["error", "always"],
+  },
 };

package/.github/workflows/node.js.yml

@@ -20,7 +20,7 @@ jobs:
         # Keep in sync with netlify.toml
         node-version: 20.x
     - name: Cache node_modules
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       id: cache
       with:
         # Caching node_modules isn't recommended because it can break across
@@ -42,13 +42,13 @@ jobs:
       with:
         node-version: 20.x
     - name: Load node_modules from cache
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         # Use node_modules from previous jobs
         path: 'node_modules'
         key: ${{ runner.os }}-node-20-${{ hashFiles('package*.json') }}
     - name: Cache BookReader/
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       id: build-cache
       with:
         # Cache the build files so we don't have to build twice for e2e tests
@@ -66,14 +66,16 @@ jobs:
       with:
         node-version: 20.x
     - name: Load node_modules from cache
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         # Use node_modules from previous jobs
         path: 'node_modules'
         key: ${{ runner.os }}-node-20-${{ hashFiles('package*.json') }}
     - run: npm run lint
     - run: npm run test
-    - run: npm run codecov
+    - uses: codecov/codecov-action@v4
+      with:
+        token: ${{ secrets.CODECOV_TOKEN }}
     
   e2e-tests:
     needs: build
@@ -84,13 +86,13 @@ jobs:
       with:
         node-version: 20.x
     - name: Load node_modules from cache
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         # Use node_modules from previous jobs
         path: 'node_modules'
         key: ${{ runner.os }}-node-20-${{ hashFiles('package*.json') }}
     - name: Load BookReader/ from cache
-      uses: actions/cache@v3
+      uses: actions/cache@v4
       with:
         # Cache the build files so we don't have to build twice for e2e tests
         path: 'BookReader'

package/BookReader/BookReader.css

@@ -698,10 +698,10 @@ i.BRicon {
 }
 
 .BRicon.read {
+  transition-property: background-color, border, background-position;
   width: 34px;
   height: 34px;
   background-position: -203px center;
-  transition-property: background-color, border, background-position;
 }
 @media (max-width: 800px) {
   .BRicon.read {
@@ -712,10 +712,10 @@ i.BRicon {
 }
 
 .BRicon.unread {
+  transition-property: background-color, border, background-position;
   width: 34px;
   height: 34px;
   background-position: -283px center;
-  transition-property: background-color, border, background-position;
 }
 @media (max-width: 800px) {
   .BRicon.unread {