Joseph B. McQueen
2984e8e851
The nav link race:
- Every page's header used `<a id="jobLink" href=''>` and the real
destination was only assigned later, after /info returned. An empty
href resolves to the current document URL, so clicking "Monitor Jobs"
on sendMessage before /info completed silently reloaded sendMessage.
- Fixed structurally: nav links now use static relative hrefs baked
into the HTML ("./sendMessage.html", "./monitorJobs.html"), so the
destination is correct the moment the DOM parses.
Shared UI:
- New html/css/app.css: design tokens (palette, radius, shadow, font),
sticky compact top header (bot avatar + label on the left, nav pills
in the middle, current user on the right, active-page highlight via
aria-current), card containers, form styling with focus rings,
DataTables theme overrides, status pills, modal, and responsive
breakpoints.
- New html/js/app.js: shared browser bootstrap. Parses appName from
the URL, redirects to OAuth if the id cookie is missing, fetches
/info once, populates the header, applies aria-current to the
active nav link, and invokes a per-page onReady callback with
{ info, appName }. Also exports getCookie, escapeHtml, and formatDate
helpers so each page stops shipping its own copy.
Per-page rewrites:
- sendMessage.html/.js: form now lives in a card, image preview only
shows when a file is attached, EasyMDE + VirtualSelect styled to
match the theme, submit is a primary button, confirmation modal
redesigned. All bootstrap code deleted (delegated to app.js).
- monitorJobs.html/.js: three cards (Running / Scheduled / Completed)
with themed DataTables. Completed table sorts by start time desc,
paginates, and searches; message column truncates HTML previews to
~120 chars. Empty-state text per table. `jobId` and running-row
"view" links go to jobDetail via safe relative URLs.
- jobDetail.html/.js: same shared header + card layout; summary grid,
message preview, and recipient table styled to match the new
palette.
Sanity checks:
- All 43 helper tests still pass.
- Server boots cleanly on port 3001.
- Curl of sendMessage/monitorJobs/jobDetail all return 200 with the
shared header markup.
- /CollabCentral/:app/css/app.css and /CollabCentral/:app/js/app.js
both serve 200 (shared static mount is per-bot as expected).
- No local href in any page is empty; every nav target resolves at
parse time.
- /info and requireBot 404 gate unchanged.
Co-authored-by: Cursor <cursoragent@cursor.com>
71 lines
2.6 KiB
HTML
71 lines
2.6 KiB
HTML
<!DOCTYPE html>
|
|
<html lang="en">
|
|
|
|
<head>
|
|
<meta charset="utf-8">
|
|
<meta name="viewport" content="width=device-width, initial-scale=1">
|
|
<title>CollabCentral</title>
|
|
<link id="appFavicon" rel="icon" type="image/x-icon" href="favicon.ico">
|
|
<link rel="stylesheet" href="css/app.css">
|
|
<link rel="stylesheet" href="https://cdn.datatables.net/1.13.6/css/jquery.dataTables.css">
|
|
<script src="https://code.jquery.com/jquery-1.11.2.min.js"></script>
|
|
<script src="https://cdn.datatables.net/1.13.6/js/jquery.dataTables.js"></script>
|
|
</head>
|
|
|
|
<body data-page="monitor" data-title-suffix="Job Detail">
|
|
|
|
<header class="appHeader">
|
|
<div class="appHeader__inner">
|
|
<div class="appHeader__brand">
|
|
<div id="appAvatar" class="appHeader__avatar"></div>
|
|
<div id="appLabel" class="appHeader__label">Loading…</div>
|
|
</div>
|
|
<nav class="appHeader__nav">
|
|
<a href="./sendMessage.html" data-nav="send">Send Message</a>
|
|
<a href="./monitorJobs.html" data-nav="monitor">Monitor Jobs</a>
|
|
</nav>
|
|
<div class="appHeader__spacer"></div>
|
|
<div id="appUser" class="appHeader__user"></div>
|
|
</div>
|
|
</header>
|
|
|
|
<div id="unauthorizedPanel" class="panel hidden">
|
|
<h2>You don't have access to this bot</h2>
|
|
<p>Your account isn't on the authorized list for this bot. Contact the bot's administrator.</p>
|
|
</div>
|
|
|
|
<div id="notFoundPanel" class="panel hidden">
|
|
<h2>Job not found</h2>
|
|
<p>The job you're looking for doesn't exist, has been purged, or belongs to a different bot.</p>
|
|
</div>
|
|
|
|
<div id="detailPanel" class="container hidden">
|
|
<div class="card">
|
|
<h1 class="pageTitle" id="jobTitle"></h1>
|
|
<p class="pageSubtitle" id="subtitle"></p>
|
|
|
|
<dl class="summary-grid" id="summaryGrid"></dl>
|
|
|
|
<div class="section-title">Message</div>
|
|
<div class="message-preview" id="messagePreview"></div>
|
|
|
|
<div class="section-title">Recipients (<span id="recipientCount">0</span>)</div>
|
|
<table id="recipients" class="dataTable" style="width: 100%;">
|
|
<thead>
|
|
<tr>
|
|
<th>Name</th>
|
|
<th>Email</th>
|
|
<th>Status</th>
|
|
<th>Send time</th>
|
|
<th>Error</th>
|
|
</tr>
|
|
</thead>
|
|
</table>
|
|
</div>
|
|
</div>
|
|
|
|
<script src="js/app.js"></script>
|
|
<script src="jobDetail.js"></script>
|
|
</body>
|
|
|
|
</html>
|