Skip to content

fix(TaskListV2): revert overflowX hidden that hides task text labels#1215

Merged
kevincodex1 merged 1 commit into
Gitlawb:mainfrom
evertjr:fix/revert-task-item-overflow-hidden
May 17, 2026
Merged

fix(TaskListV2): revert overflowX hidden that hides task text labels#1215
kevincodex1 merged 1 commit into
Gitlawb:mainfrom
evertjr:fix/revert-task-item-overflow-hidden

Conversation

@evertjr
Copy link
Copy Markdown
Contributor

@evertjr evertjr commented May 17, 2026

Apologies — the overflowX="hidden" I added in #1211 was meant to fix the orphaned task icons, but it clips the subject text to nothing when items are nested inside MessageResponse.

Separately, while investigating I noticed a lot of the spinner jitter, occasional mid-screen garbage characters, and other animation glitches disappear when openclaude is run under bun instead of node. The bin shebang currently uses node, and the dev script does too. From what I could see, the difference comes down to how the two runtimes handle stdout write atomicity and microtask scheduling , bun keeps frames coherent end-to-end while node tends to expose intermediate render states. This is probably why CC uses bun.

The overflowX="hidden" added in Gitlawb#1211 clips task subject text to
nothing when TaskItems are nested inside MessageResponse (the └
prefix constrains available width). The icon survives at 2 chars
but the text gets fully clipped, leaving orphaned ✓/■ without
any label.

Reverts the overflowX="hidden" portion of Gitlawb#1211.
Copy link
Copy Markdown
Collaborator

@gnanam1990 gnanam1990 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick turnaround on this. Confirmed the regression: when TaskListV2 renders nested inside MessageResponse, the " └ " prefix shrinks the available width, and overflowX="hidden" clips the subject text to zero while the 2-char icon survives — so we get orphaned / markers with no labels. Reverting just the TaskListV2 portion while keeping the spinner flexWrap="nowrap" fix from #1211 is the right call. Build, smoke, and the suite are clean locally (the two failing tests are pre-existing provider/network ones, unrelated).

One follow-up, not blocking: maxSubjectWidth/maxActivityWidth in TaskItem are still computed from the full terminal columns rather than the reduced width available inside MessageResponse, so the icon-leak case could still appear in very narrow terminals on the nested path. This revert is strictly better than the current clipped-text behavior, so let's land it and track width-aware truncation separately. Could you also split the bun-vs-node runtime note from the description into its own issue so it doesn't get lost here?

Approving.

@kevincodex1 kevincodex1 merged commit 0fba154 into Gitlawb:main May 17, 2026
2 checks passed
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.

3 participants