Repository navigation
fix: show task display name in task topbar - #20957
Conversation
| </TooltipProvider> | ||
|
|
||
| <h1 className="m-0 pl-2 text-base font-medium truncate">{task.name}</h1> | ||
| <h1 className="m-0 pl-2 text-base font-medium max-w-[520px] truncate"> |
There was a problem hiding this comment.
Just defaulting to the same value as in the Tasks list view. In the backend, we also always guarantee that the display name is only 64 characters, which should not reach this maximum.
|
When given a long task name, it causes these two buttons to touch 
|
Nice catch 👀 Addressed in: 2b64a2e with some styling changes, but not sure if it is the best way |
| </TooltipProvider> | ||
|
|
||
| <h1 className="m-0 pl-2 text-base font-medium truncate">{task.name}</h1> | ||
| <h1 className="m-0 pl-2 text-base font-medium max-w-[520px] truncate"> |
There was a problem hiding this comment.
| <h1 className="m-0 pl-2 text-base font-medium max-w-[520px] truncate"> | |
| <h1 className="m-0 pl-2 text-base font-medium max-w-[64ch] truncate"> |
We talked about this during our chat, unsure if we do this pattern elsewhere but it might look less magic to someone reading in the future
There was a problem hiding this comment.
As discussed on Slack, it looks like this doesn’t exactly meet the 64-characters, which leads to the page width expanding, which isn’t ideal 🙁
ssncferreira
added
cherry-pick/v2.29
Description
Update task topbar in task page to show display name, instead of task name.
Follow-up PR: #20918
Related to internal slack thread: https://codercom.slack.com/archives/C0992H8HGCS/p1764086497375829