From ccb41a056e1fba7570cc96bd4849947337c5c1c9 Mon Sep 17 00:00:00 2001
From: ulsklyc <108589275+ulsklyc@users.noreply.github.com>
Date: Sun, 19 Apr 2026 13:30:32 +0200
Subject: [PATCH] fix: position overdue nav-badge absolutely to avoid flex
 layout distortion (closes #56) (#57)

The nav-badge was appended as an in-flow flex child, breaking nav-item
layout: on mobile (column flex) it appeared below the label, on desktop
sidebar (row flex + justify-content:center) it was pushed far right via
margin-left:auto. Fix positions it absolutely within the nav-item and
uses DOM API instead of insertAdjacentHTML per project convention.

Co-authored-by: Ulas Kalayci <ulas.kalayci@googlemail.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 public/pages/tasks.js    | 5 ++++-
 public/styles/layout.css | 8 ++++++++
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/public/pages/tasks.js b/public/pages/tasks.js
index ad7f73c..fa1ccfb 100644
--- a/public/pages/tasks.js
+++ b/public/pages/tasks.js
@@ -828,7 +828,10 @@ function updateOverdueBadge() {
   document.querySelectorAll('[data-route="/tasks"] .nav-badge').forEach((el) => el.remove());
   if (overdue > 0) {
     document.querySelectorAll('[data-route="/tasks"]').forEach((el) => {
-      el.insertAdjacentHTML('beforeend', `<span class="nav-badge">${overdue}</span>`);
+      const badge = document.createElement('span');
+      badge.className = 'nav-badge';
+      badge.textContent = String(overdue);
+      el.appendChild(badge);
     });
   }
 }
diff --git a/public/styles/layout.css b/public/styles/layout.css
index 5075498..23343e2 100644
--- a/public/styles/layout.css
+++ b/public/styles/layout.css
@@ -193,6 +193,14 @@
   min-height: var(--target-lg);
   min-width: var(--target-lg);
   text-decoration: none;
+  position: relative;
+}
+
+.nav-item .nav-badge {
+  position: absolute;
+  top: var(--space-1);
+  right: var(--space-1);
+  margin-left: 0;
 }
 
 .nav-item:active {