From 75d777d63d0b87d0d35032063829b3b4bee086b3 Mon Sep 17 00:00:00 2001 From: Salem874 Date: Fri, 19 Jun 2026 01:44:25 +0100 Subject: [PATCH 1/5] =?UTF-8?q?chore(frontend):=20console.log=20=E2=86=92?= =?UTF-8?q?=20console.debug=20in=20App.tsx=20deep-link=20handler=20(#951)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The deep-link handler logged every received URL via `console.log()`. Other dev-only diagnostics in the codebase (`src/hooks/useTheme.ts:188-196`, `src/main.tsx:90`) use `console.debug` so the lines stay quiet at default browser log levels but remain visible when verbose mode is enabled. The line above (945-948) already shows a user-facing toast announcing the deep-link event, so the console line is purely dev-diagnostic and belongs on `console.debug`. One-line edit + inline comment pointing to the convention. Closes #951 --- src/App.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/App.tsx b/src/App.tsx index 96dd60cf..40900826 100644 --- a/src/App.tsx +++ b/src/App.tsx @@ -948,7 +948,10 @@ function App() { .getState() .addToast(`URL received from external link${codecSuffix}`, 'info'); - console.log(`Deep link received: ${url}${codec ? `, codec=${codec}` : ''}`); + // dev-only diagnostic; the user already gets the toast above + // (#951 — matches the console.debug convention in useTheme.ts + // and main.tsx, suppressed at default browser log levels) + console.debug(`Deep link received: ${url}${codec ? `, codec=${codec}` : ''}`); } catch (err) { console.error('Error in deep-link-download handler:', err); } From 9ebac6abb8eb403ae33cbaf9edf0b9a005c7df1b Mon Sep 17 00:00:00 2001 From: Salem874 Date: Fri, 19 Jun 2026 01:44:25 +0100 Subject: [PATCH 2/5] a11y(QueueItem): align Cancel/Retry tooltip with aria-label (#950) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR #945 (merged in #947) aligned `title` with `aria-label` for three QueueItem buttons (Retry-without-Wrapper, Open File, Open Folder) so the sighted hover-tooltip and the screen-reader announcement say the same thing. Two sibling buttons in the same file were missed: - Cancel button (line 617) had `title="Cancel"` but `aria-label="Cancel download"`. - Retry button (line 633) had `title="Retry"` but `aria-label="Retry download"`. Sighted-mouse users hover and see "Cancel"; screen-reader users hear "Cancel download". The longer form is the right one — it's specific (cancels THIS row, not e.g. the whole queue) and matches the verbose pattern the rest of the file establishes. Updates `title` on both buttons to match the existing `aria-label`. Two-line edit. No behavioural change. Closes #950 --- src/components/download/QueueItem.tsx | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/components/download/QueueItem.tsx b/src/components/download/QueueItem.tsx index 3bb17414..487cecc3 100644 --- a/src/components/download/QueueItem.tsx +++ b/src/components/download/QueueItem.tsx @@ -685,7 +685,9 @@ function QueueItemComponent({