From 36b05f499e8bdcd6557d507f376bd816ddd1185a Mon Sep 17 00:00:00 2001 From: Claude Date: Sat, 5 Sep 2026 16:19:56 +0000 Subject: [PATCH] fix(cuesheet): hide seconds option should not affect duration column The "hide seconds" toggle in the Cuesheet table settings was also stripping seconds from the Duration column. It should only apply to the Start/End time tags, so the Duration column now always displays its full precision (including seconds). Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01SoUSFVFdmMQN5Ro86pRvBQ --- .../cuesheet-table-elements/cuesheetColsFactory.tsx | 5 ++--- .../cuesheet-table-settings/CuesheetTableHeaderToolbar.tsx | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-elements/cuesheetColsFactory.tsx b/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-elements/cuesheetColsFactory.tsx index 9c7a8a463..af251505c 100644 --- a/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-elements/cuesheetColsFactory.tsx +++ b/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-elements/cuesheetColsFactory.tsx @@ -105,10 +105,9 @@ function MakeDuration({ getValue, row, table, column }: CuesheetCellContext) { return null; } - const { hideTableSeconds } = table.options.meta.options; const event = row.original; if (!isOntimeEvent(event)) { - return {formatDuration(getValue() as number, hideTableSeconds)}; + return {formatDuration(getValue() as number, false)}; } const { handleUpdateTimer } = table.options.meta; @@ -117,7 +116,7 @@ function MakeDuration({ getValue, row, table, column }: CuesheetCellContext) { const duration = getValue() as number; const isDurationLocked = event.timeStrategy === TimeStrategy.LockDuration; - const formattedDuration = formatDuration(duration, hideTableSeconds); + const formattedDuration = formatDuration(duration, false); const canWrite = column.columnDef.meta?.canWrite; if (!canWrite) { diff --git a/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-settings/CuesheetTableHeaderToolbar.tsx b/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-settings/CuesheetTableHeaderToolbar.tsx index bb62f3aad..501469726 100644 --- a/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-settings/CuesheetTableHeaderToolbar.tsx +++ b/apps/client/src/views/cuesheet/cuesheet-table/cuesheet-table-settings/CuesheetTableHeaderToolbar.tsx @@ -140,7 +140,7 @@ function ViewSettings({ optionsStore }: ViewSettingsProps) { defaultChecked={optionsStore.hideTableSeconds} onCheckedChange={(checked) => optionsStore.setOption('hideTableSeconds', checked)} /> - Hide seconds in table + Hide seconds in time tags