Fix: don't select work in MergeUI when 'want to read' is clicked

Stop selecting works in the MergeUI want clicking 'want to read' or any other button in that dropdown.

openlibrary/plugins/openlibrary/js/ile/utils/SelectionManager/SelectionManager.js

@@ -87,7 +87,7 @@ export default class SelectionManager {
     processClick(clickEvent) {
         // If there is text selection or the click is on a link that isn't a select handle, don't do anything
         if ((!clickEvent.shiftKey && window.getSelection()?.toString() !== '') ||
-            ($(clickEvent.target).closest('a').is('a') &&
+            ($(clickEvent.target).closest('a, button').length > 0 &&
             $(clickEvent.target).not('.ile-select-handle').length > 0)) return;
 
         const el = clickEvent.currentTarget;

tests/unit/js/SelectionManager.test.js

@@ -1,5 +1,30 @@
 import SelectionManager from '../../../openlibrary/plugins/openlibrary/js/ile/utils/SelectionManager/SelectionManager.js';
 
+function createTestElementsForProcessClick() {
+    const listItem = document.createElement('li');
+    listItem.classList.add('searchResultItem', 'ile-selectable');
+
+    const link = document.createElement('a');
+    listItem.appendChild(link);
+
+    const bookTitle = document.createElement('div');
+    bookTitle.classList.add('booktitle');
+    const bookLink = document.createElement('a');
+    bookLink.href = 'OL12345W'; // Mock href value
+    bookTitle.appendChild(bookLink);
+
+    listItem.appendChild(bookTitle);
+
+    return {listItem,link};
+}
+
+function setupSelectionManager() {
+    const sm = new SelectionManager(null, '/search');
+    sm.ile = { $statusImages: { append: jest.fn() } };
+    sm.selectedItems = { work: [] };
+    sm.updateToolbar = jest.fn();
+    return sm;
+}
 
 describe('SelectionManager', () => {
     afterEach(() => {
@@ -26,4 +51,37 @@ describe('SelectionManager', () => {
             author: [],
         });
     });
+
+
+    test('processClick - clicking on a link or button', () => {
+        const sm = setupSelectionManager();
+        const { listItem,link } = createTestElementsForProcessClick();
+
+        link.addEventListener('click', () => {
+            sm.processClick({ target: link, currentTarget: listItem });
+        });
+
+        expect(listItem.classList.contains('ile-selected')).toBe(false);
+        link.click();
+        expect(listItem.classList.contains('ile-selected')).toBe(false);
+
+        jest.clearAllMocks();
+    });
+
+    test('processClick - clicking on listItem', () => {
+        const sm = setupSelectionManager();
+        const { listItem } = createTestElementsForProcessClick();
+
+        listItem.addEventListener('click', () => {
+            sm.processClick({ target: listItem, currentTarget: listItem });
+        });
+
+        expect(listItem.classList.contains('ile-selected')).toBe(false);
+        listItem.click();
+        expect(listItem.classList.contains('ile-selected')).toBe(true);
+        listItem.click();
+        expect(listItem.classList.contains('ile-selected')).toBe(false);
+
+        jest.clearAllMocks();
+    });
 });