@tobq/boardclip 2.1.108 → 2.1.109

package/index.html

@@ -385,7 +385,8 @@ const appAdapter = {
   // Always the id-based path: window.api.copy() only takes a string, so it
   // would silently drop the clip's html/rtf. paste(id) puts every captured
   // format on the clipboard for both text and image clips.
-  copyNumpadSlot: (id) => { const it = itemById(id); if (!it) return; return window.api.paste(id); },
+  copyClip: (id) => window.api.copyClip(id),
+  copyMany: (ids) => window.api.copyMany(ids),
   activateClip: (id) => window.api.pasteAndHide(id),
   editClip: (id, _itemEl, options) => window.api.openEditor(id, { ...Core.editorFindFor(query), ...(options || {}) }),
   setClipTitle: (id, title) => revGuard(() => window.api.setClipTitle(id, title, clipRevOf(id)), false),
@@ -491,6 +492,9 @@ document.getElementById('copyDiagnostics').addEventListener('click', async () =>
 
 // --- Search ---
 searchEl.addEventListener('keydown', e => {
+  // A digit typed while a menu or popover is open belongs to it, never a
+  // quick-paste (typing "1ndex" with the + picker open pasted slot 1).
+  if (controller.isMenuOpen()) return;
   if (query === '' && !e.altKey && !e.ctrlKey && !e.metaKey && e.key >= '1' && e.key <= '9') {
     const n = +e.key;
     const nmap = numpadMap();

package/main.js

@@ -4889,9 +4889,9 @@ async function pasteAndHide(id) {
   }
 }
 
-// "Paste all": join the selected TEXT clips (newline-separated) and paste them in
-// one go. Images can't concatenate into text, so they're skipped.
-async function pasteMany(ids) {
+// "Paste all" / "Copy all": the selected TEXT clips joined (newline-separated)
+// in one go. Images can't concatenate into text, so they're skipped.
+function joinedClipText(ids) {
   const parts = [];
   for (const id of (Array.isArray(ids) ? ids : [])) {
     const item = findHistoryItem(id);
@@ -4899,18 +4899,30 @@ async function pasteMany(ids) {
     textBlobStore.hydrateTextItem(item, TEXT_DIR);
     parts.push(item.text || '');
   }
-  const text = parts.join('\n');
+  return parts.join('\n');
+}
+// The joined text onto the clipboard, never captured back as a new clip.
+function writeJoinedText(text) {
+  clipboard.writeText(text);
+  lastText = text;
+  lastImgHash = '';
+}
+async function pasteMany(ids) {
+  const text = joinedClipText(ids);
   if (!text) { hidePopup(); return; }
   pollGate = false;
   try {
-    clipboard.writeText(text);
-    lastText = text;   // don't re-capture our own joined paste as a new clip
-    lastImgHash = '';
+    writeJoinedText(text);
     await hideAndPasteForeground();
   } finally {
     pollGate = true;
   }
 }
+// "Copy all": the same text, nothing pasted; the popup stays.
+function copyMany(ids) {
+  const text = joinedClipText(ids);
+  if (text) writeJoinedText(text);
+}
 
 function validNumpadSlot(slot) {
   const slotNum = Number(slot);
@@ -6739,7 +6751,9 @@ function setupIPC() {
     },
   }));
 
-  ipcMain.handle('paste', (_, id) => {
+  // Copy without pasting: the clip onto the clipboard (the poller then sees it
+  // like any copy, so it moves to the top as a paste does).
+  ipcMain.handle('copy-clip', (_, id) => {
     const item = findHistoryItem(id);
     if (!item) return;
     setClipboardToItem(item);
@@ -6966,6 +6980,7 @@ function setupIPC() {
   ipcMain.handle('restore-items', (_, snaps) => applyRestoreItems(snaps));
   ipcMain.handle('group-assign-many', (_, targets, group, shouldHave) => applyGroupAssignMany(targets, group, shouldHave));
   ipcMain.handle('paste-many', (_, ids) => pasteMany(ids));
+  ipcMain.handle('copy-many', (_, ids) => copyMany(ids));
   ipcMain.handle('start-unify', (_, ids) => startUnify(ids));
   ipcMain.handle('unify-step', (_, sessionId, payload) => unifyStep(sessionId, payload));
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tobq/boardclip",
   "productName": "BoardClip",
-  "version": "2.1.108",
+  "version": "2.1.109",
   "description": "Cross-platform clipboard history tray app with numpad quick-paste and native cloud sync",
   "keywords": [
     "clipboard",

package/preload.js

@@ -9,7 +9,7 @@ contextBridge.exposeInMainWorld('api', {
     return () => ipcRenderer.removeListener('history-changed', listener);
   },
   getSettings: () => ipcRenderer.invoke('get-settings'),
-  paste: (id) => ipcRenderer.invoke('paste', id),
+  copyClip: (id) => ipcRenderer.invoke('copy-clip', id),
   pasteAndHide: (id) => ipcRenderer.invoke('paste-and-hide', id),
   numpadPasteAndHide: (slot) => ipcRenderer.invoke('numpad-paste-and-hide', slot),
   hidePopup: () => ipcRenderer.invoke('hide-popup'),
@@ -22,6 +22,7 @@ contextBridge.exposeInMainWorld('api', {
   restoreItems: (snaps) => ipcRenderer.invoke('restore-items', snaps),
   groupAssignMany: (targets, group, shouldHave) => ipcRenderer.invoke('group-assign-many', targets, group, shouldHave), // [{ id, rev }]
   pasteMany: (ids) => ipcRenderer.invoke('paste-many', ids),
+  copyMany: (ids) => ipcRenderer.invoke('copy-many', ids),
   startUnify: (ids) => ipcRenderer.invoke('start-unify', ids),
   deleteAll: () => ipcRenderer.invoke('delete-all'),
   pin: (id, rev) => ipcRenderer.invoke('pin', id, rev),

package/site/index.html

@@ -859,20 +859,24 @@
       deleteClip: (id) => { clips = Core.deleteItem(clips, id); },
       clearUnpinned: () => { clips = clips.filter((clip) => Core.isPinned(clip)); },
       setGroupSharedAi: (group) => { if (demoSharedGroups.has(group)) demoSharedGroups.delete(group); else demoSharedGroups.add(group); },
-      copyNumpadSlot: async (id) => {
+      // Copy without pasting (the controller says "Copied"). A page cannot paste
+      // into another app, so the demo's click (activateClip) copies too.
+      copyClip: async (id) => {
         const clip = demoAdapter.itemById(id);
         if (!clip) return;
         if (clip.type === "image") { const ok = await writeImageClipboard(clip).catch(() => false); if (!ok) await writeBrowserClipboard(clip.text || ""); }
         else await writeBrowserClipboard(clip.text || "");
+        clips = Core.touchItem(clips, id);
+        renderClips();
+      },
+      // Copy all: the app's Paste all text (text clips, one per line).
+      copyMany: async (ids) => {
+        const text = ids.map((id) => demoAdapter.itemById(id)).filter((clip) => clip && clip.type !== "image").map((clip) => clip.text || "").join("\n");
+        if (text) await writeBrowserClipboard(text);
       },
       activateClip: async (id) => {
-        const clip = demoAdapter.itemById(id);
-        if (!clip) return;
-        if (clip.type === "image") { const ok = await writeImageClipboard(clip).catch(() => false); if (!ok) await writeBrowserClipboard(clip.text || ""); }
-        else await writeBrowserClipboard(clip.text || "");
-        clips = Core.touchItem(clips, id);
+        await demoAdapter.copyClip(id);
         showToast("Copied");
-        renderClips();
       },
       editClip: (id, _itemEl, options) => openDemoEditor(id, { ...Core.editorFindFor(state.query), ...(options || {}) }),
       // The open editor's title field is authoritative for its clip (it rides

package/site/shared/clipboard-ui-core.js

@@ -898,7 +898,7 @@
   // (popup, editor, viewer), so an action looks the same wherever it appears
   // and no two actions share a glyph.
   const CLIP_ACTION_ICONS = {
-    pin: 'star', edit: 'edit_note', 'open-img': 'open_in_full', 'open-img-ext': 'open_in_new', 'save-img': 'download',
+    copy: 'content_copy', pin: 'star', edit: 'edit_note', 'open-img': 'open_in_full', 'open-img-ext': 'open_in_new', 'save-img': 'download',
     rename: 'drive_file_rename_outline', revert: 'history', 'select-similar': 'select_all', del: 'delete',
   };
   function renderClipActions(item, options) {
@@ -906,7 +906,9 @@
     const id = itemId(item) || '';
     const isImage = item && item.type === 'image';
     const I = CLIP_ACTION_ICONS;
-    let html = '';
+    // Copy (no paste) leads every row: the clip goes on the clipboard and the
+    // popup stays.
+    let html = `<button class="icon-btn" data-action="copy" data-id="${escapeHtml(id)}" title="Copy" aria-label="Copy"><span class="mi">${I.copy}</span></button>`;
     if (isImage) {
       html += `<button class="icon-btn" data-action="open-img" data-id="${escapeHtml(id)}" title="Open image"><span class="mi">${I['open-img']}</span></button><button class="icon-btn" data-action="save-img" data-id="${escapeHtml(id)}" title="Copy to Downloads"><span class="mi">${I['save-img']}</span></button>`;
     } else {
@@ -1025,6 +1027,7 @@
     const nmap = opts.numpadMap || numpadMap(items);
     const row = (action, label, cls, extra) => menuRowHtml({ icon: CLIP_ACTION_ICONS[action], label, cls, ...(extra || {}), attrs: { 'data-action': action, 'data-id': id, ...((extra && extra.attrs) || {}) } });
     let html = '<div class="bc-menu-list">';
+    html += row('copy', 'Copy');
     html += row('pin', pinned ? 'Unpin' : 'Pin');
     if (isImage) {
       if (context !== 'viewer') html += row('open-img', 'Open image');
@@ -1071,6 +1074,7 @@
     const row = (action, icon, label, cls) => menuRowHtml({ icon, label, cls, attrs: { 'data-action': action } });
     let html = '<div class="bc-menu-list">';
     html += row('bulk-paste', 'content_paste', `Paste all (${count})`);
+    html += row('bulk-copy', CLIP_ACTION_ICONS.copy, `Copy all (${count})`);
     html += submenuNodeHtml({ icon: 'sell', label: 'Add to group' }, bulkGroupTreeHtml(groups, selItems, Search.groupWeights(opts.items || [])));
     if (allText) html += row('bulk-unify', 'merge', `Unify (${count})`);
     html += MENU_SEPARATOR + row('bulk-delete', CLIP_ACTION_ICONS.del, `Delete (${count})`, 'danger');
@@ -1098,6 +1102,7 @@
     return `<span class="selection-count">${count} selected</span>
       <div class="selection-actions">
         <button class="icon-btn" type="button" data-action="bulk-paste" title="Paste all (Enter)" aria-label="Paste all"><span class="mi">content_paste</span></button>
+        <button class="icon-btn" type="button" data-action="bulk-copy" title="Copy all" aria-label="Copy all"><span class="mi">${CLIP_ACTION_ICONS.copy}</span></button>
         <button class="icon-btn" type="button" data-action="bulk-group-open" title="Group" aria-label="Group" aria-haspopup="true"><span class="mi">sell</span></button>
         ${allText ? '<button class="icon-btn" type="button" data-action="bulk-unify" title="Unify into one clip" aria-label="Unify into one clip"><span class="mi">merge</span></button>' : ''}
         <button class="icon-btn danger" type="button" data-action="bulk-delete" title="Delete selected" aria-label="Delete selected"><span class="mi">delete</span></button>
@@ -3438,8 +3443,9 @@
   //   mutate:  pin(id), numpadAssign(id,slot), numpadUnassign(slot),
   //            toggleGroup(id,group) [add-or-remove], createGroup(name),
   //            deleteGroup(group), deleteClip(id), clearUnpinned(),
-  //            setGroupSharedAi(group), copyNumpadSlot(id)
-  //   actions: activateClip(id), editClip(id,itemEl), openImage(item),
+  //            setGroupSharedAi(group)
+  //   actions: activateClip(id), copyClip(id) [the clipboard only, no paste],
+  //            copyMany(ids) [Paste all's joined text, no paste], editClip(id,itemEl), openImage(item),
   //            saveImage(item)->feedbackString|null
   //   keyboard:isSettingsOpen(), closeSettings(), hidePopup(),
   //            moveSelection(dir), activateSelected()
@@ -4500,6 +4506,14 @@
       try { if (a.pasteMany) await a.pasteMany(ids); }
       finally { clearSelection(); } // always drop the selection (app hides the popup; demo hides the bar)
     }
+    // Copy all: Paste all's joined text onto the clipboard, nothing pasted;
+    // the popup and the selection stay.
+    async function copySelection() {
+      const ids = selectionInfo().ids;
+      if (!ids.length || !a.copyMany) return;
+      await a.copyMany(ids);
+      toast('Copied');
+    }
     async function deleteIds(ids) {
       if (!ids || !ids.length) return;
       ids.forEach((id) => selectedIds.delete(id));
@@ -4709,6 +4723,16 @@
       const bulkGroupOpen = t.closest('[data-action="bulk-group-open"]');
       if (bulkGroupOpen) { event.stopPropagation(); const r = bulkGroupOpen.getBoundingClientRect(); openBulkGroupMenu(r.left, r.bottom + 2); return true; }
       if (t.closest('[data-action="bulk-paste"]')) { event.stopPropagation(); pasteSelection(); return true; }
+      if (t.closest('[data-action="bulk-copy"]')) { event.stopPropagation(); copySelection(); return true; }
+      // Copy without pasting (a row's Copy, the clip menu's Copy): the row
+      // button carries data-id, the menu root does (closest).
+      const copyBtn = t.closest('[data-action="copy"]');
+      if (copyBtn) {
+        event.stopPropagation();
+        const owner = copyBtn.dataset.id ? copyBtn : copyBtn.closest('[data-id]');
+        if (owner && owner.dataset.id) { await a.copyClip(owner.dataset.id); toast('Copied'); }
+        return true;
+      }
       if (t.closest('[data-action="bulk-unify"]')) { event.stopPropagation(); unifySelection(); return true; }
       if (t.closest('[data-action="bulk-delete"]')) { event.stopPropagation(); deleteSelection(); return true; }
       if (t.closest('[data-action="bulk-clear"]')) { event.stopPropagation(); clearSelection(); return true; }
@@ -4763,7 +4787,7 @@
       const npRemove = t.closest('.np-remove');
       if (npRemove) { event.stopPropagation(); await a.numpadUnassign(Number(npRemove.dataset.slot)); refresh(); return true; }
       const slotEl = t.closest('.np-slot.has-content');
-      if (slotEl && slotEl.dataset.slotId) { event.stopPropagation(); await a.copyNumpadSlot(slotEl.dataset.slotId); toast('Copied'); return true; }
+      if (slotEl && slotEl.dataset.slotId) { event.stopPropagation(); await a.copyClip(slotEl.dataset.slotId); toast('Copied'); return true; }
       const share = t.closest('.gp-share');
       if (share) { event.stopPropagation(); await a.setGroupSharedAi(share.dataset.group); refresh(); return true; }
       const gpDel = t.closest('.gp-del');
@@ -4996,6 +5020,9 @@
       // re-painting selection classes after a full list rebuild) + exposed for tests.
       selection: selectionInfo,
       focusedId: () => focusId,
+      // An open menu or popover owns the keys (the app's empty-search digit
+      // quick-paste must not fire under it).
+      isMenuOpen: () => menu.isOpen(),
       clearSelection,
       selectAll,
       moveFocus,