fix(rows): open on the first tap on iPhone, and round the hover highlight

On a touch screen a :hover on the tap target makes iOS spend the first tap applying the hover and only open on the second — the double-tap Kristian hit on tasks (and notes). The row hover is gated behind @media (hover: hover) now, so a touch has no hover to apply and the first tap opens. The note row's action buttons, which the hover used to reveal, are shown outright on touch instead. And the task-row hover highlight is a rounded box (border radius on the row), the same corner as a note in the list. Co-Authored-By: Claude Opus 4.8 <[email protected]>
Merge pull request #53 from kristianwind/charcoal-separators
2026-09-03 22:15:23 +02:00 · 2026-09-03 20:54:56 +02:00 · 2026-09-03 20:46:51 +02:00 · 2026-09-03 20:35:30 +02:00
35 changed files with 30 additions and 5 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;
@@ -228,13 +228,21 @@
 		align-items: flex-start;
 		padding: var(--s3) var(--s2);
 		border-bottom: 1px solid var(--line);
+		border-radius: var(--radius);
 		transition:
 			opacity var(--medium) var(--ease),
 			transform var(--medium) var(--ease);
 	}

-	.row:hover {
-		background: var(--surface);
+	/* Hover only where there is a real pointer. On a touch screen a :hover on the tap
+	   target makes iOS spend the first tap applying the hover and only open on the
+	   second — the "double-tap" on iPhone. With no hover to apply, the first tap opens
+	   the task. The rounded corners are on the row itself, so the highlight is a
+	   rounded box like a note in the list. */
+	@media (hover: hover) {
+		.row:hover {
+			background: var(--surface);
+		}
 	}

 	/* The exit: a short lift and fade, so completing something reads as the task
@@ -1406,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);
 	}

@@ -1492,8 +1495,21 @@
 		white-space: nowrap;
 	}

-	.row:hover {
-		background: var(--surface);
+	/* Pointer only. On a touch screen a :hover on the tap target costs a tap — iOS
+	   spends the first applying the hover and opens on the second. Gated, the first
+	   tap opens the note. The row's buttons are shown outright on touch below, since
+	   there is no hover to reveal them. */
+	@media (hover: hover) {
+		.row:hover {
+			background: var(--surface);
+		}
+	}
+
+	@media (hover: none) {
+		.star,
+		.archive-one {
+			opacity: 1;
+		}
 	}

 	.row.on {
Author	SHA1	Message	Date
kristianwindandClaude Opus 4.8	55b8e438ad	fix(rows): open on the first tap on iPhone, and round the hover highlight On a touch screen a :hover on the tap target makes iOS spend the first tap applying the hover and only open on the second — the double-tap Kristian hit on tasks (and notes). The row hover is gated behind @media (hover: hover) now, so a touch has no hover to apply and the first tap opens. The note row's action buttons, which the hover used to reveal, are shown outright on touch instead. And the task-row hover highlight is a rounded box (border radius on the row), the same corner as a note in the list. Co-Authored-By: Claude Opus 4.8 <[email protected]>	2026-09-03 22:15:23 +02:00
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 <[email protected]>	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