CSS: migrate links.html static inline styles to classes

- lt-divider--unifi / lt-divider-label--unifi: replace hardcoded margin and cyan label color on the UniFi switch section divider - lt-text-amber / lt-text-cyan on stat card icons and values (matches same migration done in index.html) - lt-stats-grid--mb: margin-bottom:16px on the summary stats grid - g-page-sub-aside: replaces margin-left:8px on the updated timestamp span in links and inspector page subtitle Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
CSS: extract notification panel inline styles to classes
2026-05-10 23:19:32 -04:00 · 2026-05-10 23:18:33 -04:00 · 2026-05-10 23:17:22 -04:00 · 2026-05-10 23:15:15 -04:00 · 2026-05-10 23:13:24 -04:00
5 changed files with 77 additions and 38 deletions
@@ -118,6 +118,39 @@
  padding: 0.2rem 0.45rem;
 }

+/* ── Form group modifiers ────────────────────────────────────────── */
+.lt-form-group--last { margin-bottom: 0; }
+
+/* ── Search input size variant ───────────────────────────────────── */
+.lt-search-input--sm { width: 180px; }
+
+/* ── Notification panel helpers ──────────────────────────────────── */
+.lt-notif-empty {
+  padding: 1rem;
+  font-size: 0.75rem;
+  color: var(--text-muted);
+  text-align: center;
+}
+.lt-notif-view-all {
+  width: 100%;
+  text-align: center;
+  display: block;
+  font-size: 0.72rem;
+}
+.lt-notif-dot {
+  border-radius: 50%;
+  margin-top: 4px;
+  flex-shrink: 0;
+}
+
+/* ── Divider variants ────────────────────────────────────────────── */
+.lt-divider--compact { margin: 1rem 0 0.75rem; }
+.lt-divider--unifi   { margin: 20px 0 12px; }
+.lt-divider-label--unifi { color: var(--cyan); letter-spacing: .1em; }
+
+/* ── Stats grid spacing variant ──────────────────────────────────── */
+.lt-stats-grid--mb { margin-bottom: 16px; }
+
 /* ── Topology section collapse toggle ────────────────────────────── */
 .topo-collapse-btn {
  margin-left: auto;
@@ -185,6 +218,7 @@
 .events-filter-bar .lt-input-sm { width: 220px; }
 .sev-pills          { display: flex; gap: 4px; }
 .g-page-sub { font-size: .78em; color: var(--text-muted); margin-top: 4px; }
+.g-page-sub-aside { font-size: .78em; color: var(--text-muted); margin-left: 8px; }

 /* ── Badge severity color variants (used with lt-badge) ───────────── */
 .badge-critical { color: var(--red);    border-color: var(--red);    text-shadow: var(--glow-red); }
@@ -410,6 +444,9 @@
  background: linear-gradient(to bottom, var(--cyan), var(--green));
  opacity: .7;
 }
+.topo-vc-wire--wan  { background: linear-gradient(to bottom, var(--cyan), rgba(0,212,255,.3)); opacity: .7; }
+.topo-vc-wire--10g  { background: var(--amber); opacity: .6; }
+.topo-vc-wire--mgmt { background: var(--border-color); opacity: .5; }
 /* Blurred copy of the wire for a soft glow halo */
 .topo-vc-wire::before {
  content: '';
@@ -467,6 +504,7 @@
 .topo-v2-sub   { font-size: .58em; color: var(--text-muted); letter-spacing: .02em; }
 .topo-v2-vlan  { font-size: .54em; color: var(--cyan); opacity: .75; }

+.topo-v2-host--bus { min-width: 80px; max-width: 96px; }
 .topo-v2-internet { border-color: var(--cyan);  color: var(--cyan);  text-shadow: var(--glow-cyan); box-shadow: 0 0 12px rgba(0,212,255,.12); }
 .topo-v2-router   { border-color: var(--cyan);  color: var(--cyan);  text-shadow: var(--glow-cyan); box-shadow: 0 0 12px rgba(0,212,255,.14); }
 .topo-v2-switch   { border-color: var(--amber); color: var(--amber); text-shadow: var(--glow-amber); box-shadow: 0 0 12px rgba(255,179,0,.12); }
@@ -516,6 +554,7 @@
 /* Bus rails */
 .topo-bus-section {
  width: 100%;
+  max-width: 860px;
  display: flex;
  flex-direction: column;
  align-items: stretch;
@@ -133,10 +133,10 @@
            <button type="button" class="lt-notif-panel-clear" id="lt-notif-clear-btn">Mark all read</button>
          </div>
          <div class="lt-notif-panel-list" id="lt-notif-list">
-            <div style="padding:0.75rem;font-size:0.75rem;color:var(--text-muted);text-align:center">Loading&hellip;</div>
+            <div class="lt-notif-empty">Loading&hellip;</div>
          </div>
          <div class="lt-notif-panel-footer">
-            <a href="{{ url_for('index') }}" class="lt-btn lt-btn-ghost lt-btn-sm" style="width:100%;text-align:center;display:block;font-size:0.72rem">View dashboard</a>
+            <a href="{{ url_for('index') }}" class="lt-btn lt-btn-ghost lt-btn-sm lt-notif-view-all">View dashboard</a>
          </div>
        </div>
      </div>
@@ -207,7 +207,7 @@
      </div>
      <form id="suppress-form">
        <div class="lt-modal-body">
-          <div class="lt-form-group" style="margin-bottom:12px">
+          <div class="lt-form-group">
            <label class="lt-label" for="sup-type">Target Type</label>
            <select class="lt-select" id="sup-type" name="target_type">
              <option value="host">Host (all interfaces)</option>
@@ -216,20 +216,20 @@
              <option value="all">Global Maintenance</option>
            </select>
          </div>
-          <div class="lt-form-group" id="sup-name-group" style="margin-bottom:12px">
+          <div class="lt-form-group" id="sup-name-group">
            <label class="lt-label" for="sup-name">Target Name</label>
            <input type="text" class="lt-input" id="sup-name" name="target_name" placeholder="e.g. large1">
          </div>
-          <div class="lt-form-group" id="sup-detail-group" style="margin-bottom:12px;display:none">
+          <div class="lt-form-group" id="sup-detail-group" style="display:none">
            <label class="lt-label" for="sup-detail">Interface <span class="lt-field-hint">(interface type only)</span></label>
            <input type="text" class="lt-input" id="sup-detail" name="target_detail" placeholder="e.g. enp35s0">
          </div>
-          <div class="lt-form-group" style="margin-bottom:12px">
+          <div class="lt-form-group">
            <label class="lt-label" for="sup-reason">Reason <span class="required">*</span></label>
            <input type="text" class="lt-input" id="sup-reason" name="reason"
                   placeholder="e.g. Planned switch reboot" required>
          </div>
-          <div class="lt-form-group" style="margin-bottom:0">
+          <div class="lt-form-group lt-form-group--last">
            <label class="lt-label">Duration</label>
            <div class="duration-pills">
              <button type="button" class="pill" data-duration="30">30 min</button>
@@ -258,7 +258,7 @@
        <button type="button" class="lt-modal-close" data-modal-close aria-label="Close">&#x2715;</button>
      </div>
      <div class="lt-modal-body">
-        <table class="lt-table" style="width:100%">
+        <table class="lt-table">
          <thead><tr><th>Shortcut</th><th>Action</th></tr></thead>
          <tbody>
            <tr><td>Ctrl / &#x2318; + K</td><td>Command palette</td></tr>
@@ -295,7 +295,7 @@
          </div>
          <div class="lt-field-hint" id="settings-refresh-hint"></div>
        </div>
-        <div class="lt-divider" style="margin:1rem 0 0.75rem"></div>
+        <div class="lt-divider lt-divider--compact"></div>
        <div class="lt-kv-grid">
          <span class="lt-kv-key">User</span>
          <span class="lt-kv-val lt-kv-val--cyan">{{ user.name or user.username }}</span>
@@ -455,14 +455,14 @@
        lt.notif.set(bell, unreadCount);

        if (!active.length) {
-          list.innerHTML = '<div style="padding:1rem;font-size:0.75rem;color:var(--text-muted);text-align:center">&#x2714; No active alerts</div>';
+          list.innerHTML = '<div class="lt-notif-empty">&#x2714; No active alerts</div>';
          return;
        }
        list.innerHTML = active.slice(0, 25).map(function(e) {
          var isUnread = toMs(e.last_seen) > readBefore;
          var dotColor = SEV_DOT[e.severity] || 'var(--text-muted)';
          return '<div class="lt-notif-item' + (isUnread ? ' lt-notif-item--unread' : '') + '">' +
-            '<div class="lt-notif-dot' + (isUnread ? '' : ' lt-notif-dot--read') + '" style="background:' + dotColor + ';border-radius:50%;margin-top:4px"></div>' +
+            '<div class="lt-notif-dot' + (isUnread ? '' : ' lt-notif-dot--read') + '" style="background:' + dotColor + '"></div>' +
            '<div class="lt-notif-item-body">' +
              '<div class="lt-notif-item-title">' + esc(e.target_name) + (e.target_detail ? ' &middot; ' + esc(e.target_detail) : '') + '</div>' +
              '<div class="lt-notif-item-time">' + esc(e.event_type.replace(/_/g,' ')) + ' &middot; ' + fmtAgo(e.last_seen) + '</div>' +
@@ -486,7 +486,7 @@
            }
          })
          .catch(function() {
-            if (andRender) list.innerHTML = '<div style="padding:0.75rem;font-size:0.75rem;color:var(--text-muted);text-align:center">Could not load</div>';
+            if (andRender) list.innerHTML = '<div class="lt-notif-empty">Could not load</div>';
          });
      }

@@ -31,32 +31,32 @@
  <div class="lt-stat-card{% if summary.critical %} lt-stat-card--alert{% endif %}"
       id="stat-critical" role="button" tabindex="0"
       data-stat-filter="critical" aria-label="{{ summary.critical or 0 }} critical alerts">
-    <span class="lt-stat-icon" aria-hidden="true" style="color:var(--red);text-shadow:var(--glow-red)">●</span>
+    <span class="lt-stat-icon lt-text-red" aria-hidden="true">●</span>
    <div class="lt-stat-info">
-      <span class="lt-stat-value" id="stat-critical-val" style="color:var(--red)">{{ summary.critical or 0 }}</span>
+      <span class="lt-stat-value lt-text-red" id="stat-critical-val">{{ summary.critical or 0 }}</span>
      <span class="lt-stat-label">Critical</span>
    </div>
  </div>
  <div class="lt-stat-card"
       id="stat-warning" role="button" tabindex="0"
       data-stat-filter="warning" aria-label="{{ summary.warning or 0 }} warning alerts">
-    <span class="lt-stat-icon" aria-hidden="true" style="color:var(--amber)">●</span>
+    <span class="lt-stat-icon lt-text-amber" aria-hidden="true">●</span>
    <div class="lt-stat-info">
-      <span class="lt-stat-value" id="stat-warning-val" style="color:var(--amber)">{{ summary.warning or 0 }}</span>
+      <span class="lt-stat-value lt-text-amber" id="stat-warning-val">{{ summary.warning or 0 }}</span>
      <span class="lt-stat-label">Warning</span>
    </div>
  </div>
  <div class="lt-stat-card" id="stat-hosts" aria-label="Monitored hosts">
-    <span class="lt-stat-icon" aria-hidden="true" style="color:var(--cyan)">⬡</span>
+    <span class="lt-stat-icon lt-text-cyan" aria-hidden="true">⬡</span>
    <div class="lt-stat-info">
-      <span class="lt-stat-value" id="stat-hosts-val" style="color:var(--cyan)">{{ snapshot.hosts | length }}</span>
+      <span class="lt-stat-value lt-text-cyan" id="stat-hosts-val">{{ snapshot.hosts | length }}</span>
      <span class="lt-stat-label">Hosts</span>
    </div>
  </div>
  <div class="lt-stat-card" id="stat-resolved" aria-label="{{ recent_resolved | length }} alerts resolved in last 24 hours">
-    <span class="lt-stat-icon" aria-hidden="true" style="color:var(--green);text-shadow:var(--glow)">✔</span>
+    <span class="lt-stat-icon lt-text-green" aria-hidden="true">✔</span>
    <div class="lt-stat-info">
-      <span class="lt-stat-value" id="stat-resolved-val" style="color:var(--green)">{{ recent_resolved | length }}</span>
+      <span class="lt-stat-value lt-text-green" id="stat-resolved-val">{{ recent_resolved | length }}</span>
      <span class="lt-stat-label">Resolved 24h</span>
    </div>
  </div>
@@ -188,9 +188,9 @@
      </div>
    </div>

-    <!-- WAN wire: cyan → green gradient, labeled -->
+    <!-- WAN wire: cyan → WAN gradient -->
    <div class="topo-vc">
-      <div class="topo-vc-wire" style="background:linear-gradient(to bottom,var(--cyan),rgba(0,212,255,.3)); opacity:.7;"></div>
+      <div class="topo-vc-wire topo-vc-wire--wan"></div>
      <span class="topo-vc-label">WAN · 10G SFP+</span>
    </div>

@@ -208,7 +208,7 @@

    <!-- UDM-Pro → USW-Agg (10G SFP+) -->
    <div class="topo-vc">
-      <div class="topo-vc-wire" style="background:var(--amber);opacity:.6;"></div>
+      <div class="topo-vc-wire topo-vc-wire--10g"></div>
      <span class="topo-vc-label">10G SFP+</span>
    </div>

@@ -227,7 +227,7 @@

    <!-- USW-Agg → Pro 24 PoE (10G trunk) -->
    <div class="topo-vc">
-      <div class="topo-vc-wire" style="background:var(--amber);opacity:.6;"></div>
+      <div class="topo-vc-wire topo-vc-wire--10g"></div>
      <span class="topo-vc-label">10G trunk</span>
    </div>

@@ -246,14 +246,14 @@

    <!-- Pro 24 PoE → host bus section -->
    <div class="topo-vc">
-      <div class="topo-vc-wire" style="background:var(--border-color);opacity:.5;"></div>
+      <div class="topo-vc-wire topo-vc-wire--mgmt"></div>
    </div>

    <!-- ══════════════════════════════════════════════════════════════
         TIER 4 connecting bus – two rails (10G green + 1G amber dashed)
         showing dual-homing for all 6 servers
         ══════════════════════════════════════════════════════════ -->
-    <div class="topo-bus-section" style="max-width:860px;">
+    <div class="topo-bus-section">

      <!-- 10G storage bus (Agg → VLAN90) -->
      <div class="topo-bus-10g">
@@ -286,8 +286,8 @@
            <div class="topo-v2-wire-1g"  data-host="{{ hname }}" title="1G → Pro 24 PoE"></div>
          </div>
          <!-- host box -->
-          <div class="topo-v2-node topo-v2-host topo-host topo-v2-status-{{ st }}{{ ' topo-v2-offrack' if off_rack else '' }}"
-               data-host="{{ hname }}" style="min-width:80px; max-width:96px;">
+          <div class="topo-v2-node topo-v2-host topo-host topo-v2-status-{{ st }}{{ ' topo-v2-offrack' if off_rack else '' }} topo-v2-host--bus"
+               data-host="{{ hname }}">
            <span class="topo-v2-icon">▣</span>
            <span class="topo-v2-label">{{ hlabel }}</span>
            <span class="topo-v2-sub">{{ hsub }}</span>
@@ -315,8 +315,8 @@
  <div class="lt-toolbar" id="host-toolbar">
    <div class="lt-toolbar-left">
      <div class="lt-search">
-        <input type="search" class="lt-input lt-search-input" id="host-search"
-               placeholder="Filter hosts…" autocomplete="off" style="width:180px">
+        <input type="search" class="lt-input lt-search-input lt-search-input--sm" id="host-search"
+               placeholder="Filter hosts…" autocomplete="off">
      </div>
    </div>
  </div>
@@ -8,7 +8,7 @@
    <h1 class="lt-page-title">Network Inspector</h1>
    <p class="g-page-sub">
      Visual switch chassis diagrams. Click a port to see detailed stats and LLDP path debug.
-      <span id="inspector-updated" style="margin-left:8px"></span>
+      <span id="inspector-updated" class="g-page-sub-aside"></span>
    </p>
  </div>
 </div>
@@ -8,7 +8,7 @@
    <h1 class="lt-page-title">Link Debug</h1>
    <p class="g-page-sub">
      Per-interface stats: speed, duplex, SFP optical levels, TX/RX rates, errors, and carrier changes.
-      <span id="links-updated" style="margin-left:8px"></span>
+      <span id="links-updated" class="g-page-sub-aside"></span>
    </p>
  </div>
 </div>
@@ -358,7 +358,7 @@ function renderUnifiSwitches(unifiSwitches, dataUpdated) {
      </div>`;
  }).join('');

-  return `<div class="lt-divider" style="margin:20px 0 12px"><span class="lt-divider-label" style="color:var(--cyan);letter-spacing:.1em">UNIFI SWITCH PORTS</span></div>${html}`;
+  return `<div class="lt-divider lt-divider--unifi"><span class="lt-divider-label lt-divider-label--unifi">UNIFI SWITCH PORTS</span></div>${html}`;
 }

 // ── Panel collapse / expand ───────────────────────────────────────
@@ -412,18 +412,18 @@ function buildLinkSummary(hosts, unifiSwitches) {
  const downCardCls = allDown  > 0 ? ' lt-stat-card--alert' : '';
  const poeCard = totalPoe > 0 ? `
    <div class="lt-stat-card">
-      <span class="lt-stat-icon" aria-hidden="true" style="color:var(--amber)">⚡</span>
+      <span class="lt-stat-icon lt-text-amber" aria-hidden="true">⚡</span>
      <div class="lt-stat-info">
-        <span class="lt-stat-value" style="color:var(--amber)">${totalPoe.toFixed(1)}</span>
+        <span class="lt-stat-value lt-text-amber">${totalPoe.toFixed(1)}</span>
        <span class="lt-stat-label">PoE Load (W)</span>
      </div>
    </div>` : '';
  return `
-    <div class="lt-stats-grid" style="margin-bottom:16px">
+    <div class="lt-stats-grid lt-stats-grid--mb">
      <div class="lt-stat-card">
-        <span class="lt-stat-icon" aria-hidden="true" style="color:var(--cyan)">⬡</span>
+        <span class="lt-stat-icon lt-text-cyan" aria-hidden="true">⬡</span>
        <div class="lt-stat-info">
-          <span class="lt-stat-value" style="color:var(--cyan)">${allTotal}</span>
+          <span class="lt-stat-value lt-text-cyan">${allTotal}</span>
          <span class="lt-stat-label">Interfaces</span>
        </div>
      </div>
Author	SHA1	Message	Date
jared	4dd7fc16f3	CSS: migrate links.html static inline styles to classes Lint / Python (flake8) (push) Successful in 41s Details Lint / JS (eslint) (push) Successful in 7s Details Security / Python Security (bandit) (push) Successful in 40s Details Test / Python Tests (pytest) (push) Successful in 48s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 4s Details - lt-divider--unifi / lt-divider-label--unifi: replace hardcoded margin and cyan label color on the UniFi switch section divider - lt-text-amber / lt-text-cyan on stat card icons and values (matches same migration done in index.html) - lt-stats-grid--mb: margin-bottom:16px on the summary stats grid - g-page-sub-aside: replaces margin-left:8px on the updated timestamp span in links and inspector page subtitle Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:19:32 -04:00
jared	0b33589106	CSS: extract notification panel inline styles to classes Lint / Python (flake8) (push) Successful in 40s Details Lint / JS (eslint) (push) Successful in 8s Details Security / Python Security (bandit) (push) Successful in 1m7s Details Test / Python Tests (pytest) (push) Successful in 1m42s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 3s Details - lt-notif-empty: replaces all hardcoded padding/font/color/align on the empty-state and loading/error text in the notification bell panel - lt-notif-view-all: replaces width/text-align/display/font-size inline style on the 'View dashboard' footer link - lt-notif-dot: moves border-radius:50%;margin-top from inline style (only background color remains inline, which is dynamic per-severity) - Initial 'Loading…' text in the panel HTML uses lt-notif-empty Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:18:33 -04:00
jared	ca4bcef26c	CSS: replace remaining inline color/size styles with TDS utilities Lint / Python (flake8) (push) Successful in 57s Details Lint / JS (eslint) (push) Successful in 7s Details Security / Python Security (bandit) (push) Successful in 53s Details Test / Python Tests (pytest) (push) Successful in 1m13s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 9s Details - Stat card icons and values: style="color:var(--red)" etc replaced with lt-text-red, lt-text-amber, lt-text-cyan, lt-text-green (defined in base.css with both color and glow-shadow) - Host search input: style="width:180px" extracted to .lt-search-input--sm - base.html: suppress modal form groups use lt-form-group--last for last item (already committed); lt-divider--compact applied to settings divider Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:17:22 -04:00
jared	15120a280f	CSS: remove remaining fixable inline styles across templates Lint / Python (flake8) (push) Successful in 48s Details Lint / JS (eslint) (push) Successful in 7s Details Security / Python Security (bandit) (push) Successful in 42s Details Test / Python Tests (pytest) (push) Successful in 54s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 3s Details - Suppress modal form groups: strip margin-bottom:12px (lt-form-group already has margin-bottom in TDS); use lt-form-group--last for the final group where zero margin is needed - Keyboard shortcuts table: remove width:100% (lt-table is already full- width in base.css) - Settings divider: replace style=margin override with .lt-divider--compact - Topology bus section: move max-width:860px into .topo-bus-section rule Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:15:15 -04:00
jared	906869f425	CSS: convert all topology inline styles to modifier classes Lint / Python (flake8) (push) Successful in 41s Details Lint / JS (eslint) (push) Successful in 9s Details Security / Python Security (bandit) (push) Successful in 1m0s Details Test / Python Tests (pytest) (push) Successful in 55s Details Lint / Notify on failure (push) Has been skipped Details Lint / Deploy (push) Successful in 4s Details Replace style= attributes on topology wire/node elements with semantic modifier classes: - topo-vc-wire--wan, --10g, --mgmt (wire colour semantics in CSS) - topo-v2-host--bus (bus-section node size constraint) - topo-legend-item--offrack already done in prior commit Zero inline styles remain in the topology section. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-05-10 23:13:24 -04:00