From 0c1859dc43df78725017e363f7855b566de2ff19 Mon Sep 17 00:00:00 2001 From: Adorian Doran Date: Fri, 2 Jan 2026 20:59:51 +0200 Subject: [PATCH 01/19] style/note splits: highlight the active split only in a multi-split view --- apps/client/src/stylesheets/theme-next/shell.css | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/apps/client/src/stylesheets/theme-next/shell.css b/apps/client/src/stylesheets/theme-next/shell.css index cd016eee1..f81e8fcf7 100644 --- a/apps/client/src/stylesheets/theme-next/shell.css +++ b/apps/client/src/stylesheets/theme-next/shell.css @@ -1260,13 +1260,15 @@ body.layout-horizontal #rest-pane > .classic-toolbar-widget { padding-top: 2px; background-color: var(--note-split-background-color, var(--main-background-color)); transition: border-color 250ms ease-in; - border: 1px solid transparent; - - &.active { - border-color: var(--link-selection-outline-color); - } + border: 2px solid transparent; } +/* The active split in a multi-split view */ +#center-pane > .split-note-container-widget:has(> .note-split.visible ~ .note-split.visible) > .note-split.active { + border-color: var(--link-selection-outline-color); +} + + body:not(.background-effects) #center-pane .note-split { animation: note-entrance 100ms linear; } From e161ffce57b68ac5eeebf1af77d374107900207d Mon Sep 17 00:00:00 2001 From: Elian Doran Date: Fri, 2 Jan 2026 21:20:29 +0200 Subject: [PATCH 02/19] fix(client/pdf): not always focusing on click --- .../src/widgets/type_widgets/file/Pdf.tsx | 25 +++++-------------- 1 file changed, 6 insertions(+), 19 deletions(-) diff --git a/apps/client/src/widgets/type_widgets/file/Pdf.tsx b/apps/client/src/widgets/type_widgets/file/Pdf.tsx index c915a0a5d..6c1512d87 100644 --- a/apps/client/src/widgets/type_widgets/file/Pdf.tsx +++ b/apps/client/src/widgets/type_widgets/file/Pdf.tsx @@ -151,25 +151,6 @@ export default function PdfPreview({ note, blob, componentId, noteContext }: { } }, [ blob ]); - // Trigger focus when iframe content is clicked (iframe focus doesn't bubble) - useEffect(() => { - const iframe = iframeRef.current; - if (!iframe) return; - - const handleIframeClick = () => { - if (noteContext.ntxId) { - appContext.tabManager.activateNoteContext(noteContext.ntxId); - } - }; - - // Listen for clicks on the iframe's content window - const iframeDoc = iframe.contentWindow?.document; - if (iframeDoc) { - iframeDoc.addEventListener('click', handleIframeClick); - return () => iframeDoc.removeEventListener('click', handleIframeClick); - } - }, [ iframeRef.current?.contentWindow, noteContext ]); - return (historyConfig &&