Skip to content

fix: make timestamp toggle text dynamic in command list (resolves #5106)#5108

Merged
rekram1-node merged 4 commits into
anomalyco:devfrom
ariane-emory:feat/dynamic-toggle-timestamps-text
Dec 6, 2025
Merged

fix: make timestamp toggle text dynamic in command list (resolves #5106)#5108
rekram1-node merged 4 commits into
anomalyco:devfrom
ariane-emory:feat/dynamic-toggle-timestamps-text

Conversation

@ariane-emory

Copy link
Copy Markdown
Contributor

Change 'Toggle timestamps' to dynamically show either 'Show timestamps' or 'Hide timestamps' based on current state, matching the behavior of thinking blocks and tool details toggles.

Fixes #5106

Change 'Toggle timestamps' to dynamically show 'Show timestamps' or 'Hide timestamps'
based on current state, matching the behavior of thinking blocks and tool details toggles.

Fixes anomalyco#5106
@rekram1-node

Copy link
Copy Markdown
Collaborator

I think we wanted the commands to be more discoverable by keeping the consistent naming, but lemme ask

@ariane-emory

ariane-emory commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author

I see. For the tool_details and thinking blocks commands, I'd went with the hide/show strategy because that way the command_list item name could serve as the indicator of the current state of the toggle (for lack of any other indicator). From that perspective, this would be making the time stamp toggle more consistent with the tool_details/thinking block.

Of course, if y'all prefer consistency in the other direction (no hide/show for any of 'em) I could certainly make that happen too - although that'd leave the user with no reliable way to see what the current state of some of these toggles is, which seems like a downside. Just let me know what the team's preferences are once you've decided. ;)

@ariane-emory
ariane-emory deleted the feat/dynamic-toggle-timestamps-text branch December 6, 2025 05:47
@ariane-emory
ariane-emory restored the feat/dynamic-toggle-timestamps-text branch December 6, 2025 06:41
@ariane-emory ariane-emory reopened this Dec 6, 2025
@rekram1-node
rekram1-node merged commit 429aa24 into anomalyco:dev Dec 6, 2025
3 of 4 checks passed
@ariane-emory
ariane-emory deleted the feat/dynamic-toggle-timestamps-text branch December 6, 2025 22:15
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
AIALRA-0 pushed a commit to AIALRA-0/opencode-turn-engine that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX]: Dynamically switch command_list item text for toggling timestamp display based on its current state

2 participants