theirongolddev/session-viewer
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Polish: simplify formatTimestamp and tone down export button

Browse Source 
Replace try/catch with isNaN guard in the HTML exporter's
formatTimestamp, matching the same cleanup applied client-side.

Downgrade the export button from btn-primary to btn-secondary so it
doesn't compete visually with the main content area. The primary blue
gradient was overly prominent for a utility action.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Taylor Eernisse
2026-01-30 09:26:35 -05:00
parent c0e4158b77
commit bba678568a
2 changed files with 8 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/client/components/ExportButton.tsx
View File

@@ -65,7 +65,7 @@ export function ExportButton({
btn btn-sm flex-shrink-0 gap-1.5 transition-all duration-200 btn btn-sm flex-shrink-0 gap-1.5 transition-all duration-200
${state === "success" ? "text-white shadow-glow-success" : ""} ${state === "success" ? "text-white shadow-glow-success" : ""}
${state === "error" ? "text-white" : ""} ${state === "error" ? "text-white" : ""}
${state === "idle" || state === "exporting" ? "btn-primary" : ""} ${state === "idle" || state === "exporting" ? "btn-secondary" : ""}
`} `}
style={ style={
state === "success" state === "success"

17
src/server/services/html-exporter.ts
View File

@@ -170,16 +170,13 @@ function renderMarkdown(text: string): string {
} }
function formatTimestamp(ts: string): string { function formatTimestamp(ts: string): string {
try { const d = new Date(ts);
const d = new Date(ts); if (isNaN(d.getTime())) return "";
return d.toLocaleTimeString(undefined, { return d.toLocaleTimeString(undefined, {
hour: "2-digit", hour: "2-digit",
minute: "2-digit", minute: "2-digit",
second: "2-digit", second: "2-digit",
}); });
} catch {
return "";
}
} }
function getHighlightCss(): string { function getHighlightCss(): string {