Merge pull request #53 from kristianwind/charcoal-separators

Charcoal: lighter day separators in the note list
fix(theme): lighter day separators in Charcoal
2026-09-03 20:54:56 +02:00 · 2026-09-03 20:46:51 +02:00 · 2026-09-03 20:35:30 +02:00 · 2026-09-03 20:27:28 +02:00
2 changed files with 18 additions and 3 deletions
@@ -242,6 +242,7 @@
 	--note-on-ink: #1e1e1e; /* dark text on the gold */
 	--nav-active-ink: #1e1e1e; /* dark text on the amber menu item */
 	--heading-ink: #8a8e90; /* PROJEKTER and the group names — quiet grey */
+	--note-group-bg: #2d3133; /* day separators take the menu's resting grey, not the darker ground */

 	--p1: #cf6b6b;
 	--p2: #d69f5c;
@@ -710,7 +710,7 @@
 <!-- `has-note` drives the phone layout: one column, showing the list until a note
     is opened and the editor once one is. On a wide screen both are always up and
     the class does nothing. -->
-<div class="notes" class:has-note={selected}>
+<div class="notes" class:has-note={selectedId}>
 	<aside>
 		<div class="head">
 			<h1>{t('notes.title')}</h1>
@@ -919,10 +919,15 @@
 	</aside>

 	<section class="editor">
-		{#if selected}
+		{#if selectedId}
+			<!-- Keyed on the id, which is set the instant a row is tapped — not on the
+			     loaded note, which arrives a moment later. On a phone that moment was the
+			     whole bug: the pane did not switch until the body had loaded, so the first
+			     tap looked like nothing and people tapped again. -->
 			<!-- Phone only (hidden on a wide screen by CSS): the way back to the list,
 			     since there the editor fills the screen and the list is not beside it. -->
 			<button class="back" onclick={() => open(null)}>‹ {t('notes.title')}</button>
+			{#if selected}
 			<NoteEditor
 				note={selected}
 				notes={notes}
@@ -1056,6 +1061,12 @@
 					{/if}
 				</div>
 			</footer>
+			{:else}
+				<!-- Chosen, but its full text is still loading. Keep the pane (so the tap
+				     felt like it landed) and stay quiet — the "pick a note" line is for
+				     when nothing is chosen at all, not for this half-second. -->
+				<p class="empty" aria-busy="true"></p>
+			{/if}
 		{:else}
 			<p class="empty">{t('notes.pickOne')}</p>
 		{/if}
@@ -1395,7 +1406,10 @@
 		top: 0;
 		z-index: 1;
 		margin: 0 0 var(--s1);
-		background: var(--ground);
+		/* --ground everywhere but Kul, where the ground is the darkest thing on screen
+		   and a day separator on it reads as a dark bar. There it takes the menu's own
+		   resting colour instead, via --note-group-bg. */
+		background: var(--note-group-bg, var(--ground));
 		border-radius: var(--radius);
 	}
Author	SHA1	Message	Date
Kristian Wind	1607a22f0c	Merge pull request #53 from kristianwind/charcoal-separators CI / OpenAPI is valid (push) Successful in 11s Details CI / End-to-end (push) Successful in 9m56s Details CI / Docker image builds (push) Successful in 2m2s Details Release / Build and publish image (push) Failing after 5s Details CI / Go (push) Failing after 15m10s Details Charcoal: lighter day separators in the note list	2026-09-03 20:54:56 +02:00
kristianwindandClaude Opus 4.8	8a5dfef41a	fix(theme): lighter day separators in Charcoal In Charcoal the note-list day separators used --ground, the darkest colour in the theme, so they read as dark bars on the lighter list. They take the menu's own resting grey (#2d3133) now — the colour "Projekter" sits on — via a --note-group-bg fallback token, so every other theme keeps --ground. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2026-09-03 20:46:51 +02:00
Kristian Wind	1a610eaac4	Merge pull request #52 from kristianwind/notes-single-tap CI / OpenAPI is valid (push) Successful in 11s Details CI / End-to-end (push) Successful in 10m25s Details CI / Docker image builds (push) Successful in 1m58s Details Release / Build and publish image (push) Failing after 6s Details CI / Go (push) Failing after 15m11s Details Notes: open on the first tap, not the second	2026-09-03 20:35:30 +02:00
kristianwindandClaude Opus 4.8	c2d4ce0179	fix(notes): open a note on the first tap, not the second On a phone the list/editor switch keyed on the loaded note, which arrives a moment after the tap — so the first tap looked like nothing happened and people tapped again. It keys on the selected id now, set the instant a row is tapped, so the pane switches at once; the back button is there immediately and the body fills in when it arrives, with a quiet pane rather than the "pick a note" prompt in the half-second between. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2026-09-03 20:27:28 +02:00