Skip to content

Commit bab34fa

Browse files
committed
chore(logs): remove extraneous comments
1 parent bb40841 commit bab34fa

2 files changed

Lines changed: 0 additions & 7 deletions

File tree

apps/sim/app/workspace/[workspaceId]/logs/components/log-details/log-details.tsx

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -302,8 +302,6 @@ export const LogDetails = memo(function LogDetails({
302302
const [prevLogId, setPrevLogId] = useState(log?.id)
303303
const [copiedRunId, setCopiedRunId] = useState(false)
304304

305-
// Reset tab to overview when the selected log changes so the user never gets
306-
// trapped on the Trace tab (which would suppress arrow-key log navigation).
307305
if (prevLogId !== log?.id) {
308306
setPrevLogId(log?.id)
309307
setActiveTab('overview')
@@ -345,10 +343,6 @@ export const LogDetails = memo(function LogDetails({
345343

346344
const resolvedTab: LogDetailsTab = activeTab === 'trace' && !showTraceTab ? 'overview' : activeTab
347345

348-
// When the trace tab disappears while it is active (e.g. data not yet loaded), resolvedTab
349-
// falls back to 'overview'. Notify the parent inline so it sees the corrected value without
350-
// waiting for an extra render cycle. The user-click path calls onActiveTabChange directly in
351-
// onValueChange below.
352346
const prevResolvedTabRef = useRef<LogDetailsTab>(resolvedTab)
353347
if (prevResolvedTabRef.current !== resolvedTab) {
354348
prevResolvedTabRef.current = resolvedTab

apps/sim/app/workspace/[workspaceId]/logs/logs.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -706,7 +706,6 @@ export default function Logs() {
706706
const handleKeyDown = (e: KeyboardEvent) => {
707707
const tag = (e.target as HTMLElement)?.tagName
708708
if (tag === 'INPUT' || tag === 'TEXTAREA') return
709-
// When the trace tab is active, arrow keys belong to TraceView's span navigator.
710709
if (activeLogTabRef.current === 'trace') return
711710
const currentLogs = logsRef.current
712711
const currentIndex = selectedLogIndexRef.current

0 commit comments

Comments
 (0)