Bulletproofing discontinued book series merging => no matching book n…

…umber, append the list Related to: adfc49a + I mage the "not in library" indicator icon a bit smaller (in the gallery → series subpage)
joonaspaakko · Aug 20, 2023 · 8980036 · 8980036
1 parent b8208f2
commit 8980036
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 5 deletions.
diff --git a/src/content-script/content-script-app.vue b/src/content-script/content-script-app.vue
@@ -267,8 +267,17 @@ export default {
                   const newAddition = _.pick(book, [ 'titleShort', 'title', 'cover', 'bookNumbers', 'asin' ]);
                   newAddition.bookNumbers = series.bookNumbers.join(', ');
                   const firstNumber = _.first(series.bookNumbers);
-                  const targetIndex = _.findLastIndex(foundSeries.allBooks, { bookNumbers: firstNumber });
-                  foundSeries.allBooks.splice(targetIndex+1, 0, newAddition);
+                  const targetIndex = _.findLastIndex(foundSeries.allBooks, ( book ) => {
+                    return book.bookNumbers == firstNumber || book.bookNumbers == newAddition.bookNumbers;
+                  });
+                  
+                  // Found a book with the same number
+                  if ( targetIndex > -1 ) {
+                    foundSeries.allBooks.splice(targetIndex+1, 0, newAddition);
+                  }
+                  else {
+                    foundSeries.allBooks.push(newAddition);
+                  }
                 }
               });
             }

diff --git a/src/gallery/_components/alePages/gallery-root/gallery-grid-view/gallery-book.vue b/src/gallery/_components/alePages/gallery-root/gallery-grid-view/gallery-book.vue
@@ -522,9 +522,9 @@ div.not-in-library {
     display: flex;
     div {
       background: rgba(#000, .55);
-      width: 50%;
-      height: 50%;
-      font-size: 140%;
+      width: 25%;
+      height: 25%;
+      font-size: 120%;
     }
   }
 }