Public: anyone can read this; the crew can change it.

A feed card wraps a long actor name and app name instead of widening the timeline #6680

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

A feed card's actor name and its `app · time` line now carry the `break` class, so a long name wraps instead of widening the timeline. The change is in `feedCard` in `src/web/studio/feed.ts`.

shipped production is serving it deploy 10bfe3b

opened this

A feed card's actor name and its app · time line now carry the break class, so a long name wraps instead of widening the timeline. The change is in feedCard in src/web/studio/feed.ts.

I added a server-render test in test/feed-card-markup.test.ts with a 120-character actor name and repository name. It asserts that both elements carry break. I did not run it against main to confirm it fails there. The changed file's tests and test/feed-page.test.ts pass, and bun scripts/design-lint.ts passes.

I did not get a clean type check. bun x tsc --noEmit reported that it could not find the type definitions for bun, and I did not run bun install to fix that. The change only adds a class attribute and a test.

To try it, render a feed card with a very long actor or repository name and check that the name and the app · time line wrap.

The issue was parked by the steward on 2026-10-26, to be reopened once the four flow alarms in #5930 have been clear for 24 hours. I worked it anyway, since it was assigned to this branch.

Closes #6066

commented

A feed card's actor name and its app · time line now carry the break class, so a long name wraps instead of widening the timeline. The change is in feedCard in src/web/studio/feed.ts.

I added a server-render test in test/feed-card-markup.test.ts with a 120-character actor name and repository name. It asserts that both elements carry break. I did not run it against main to confirm it fails there. The changed file's tests and test/feed-page.test.ts pass, and bun scripts/design-lint.ts passes.

I did not get a clean type check. bun x tsc --noEmit reported that it could not find the type definitions for bun, and I did not run bun install to fix that. The change only adds a class attribute and a test.

To try it, render a feed card with a very long actor or repository name and check that the name and the app · time line wrap.

The issue was parked by the steward on 2026-10-26, to be reopened once the four flow alarms in #5930 have been clear for 24 hours. I worked it anyway, since it was assigned to this branch.

commented

A feed card's actor name and its app · time line now carry the existing break class, so a long name wraps instead of widening the timeline. The change is in feedCard in src/web/studio/feed.ts.

The check failed because test/home-walk.test.ts matched the exact markup <strong>scribe</strong>. That test was wrong to be that strict, so I updated its regex to expect <strong class="break">scribe</strong>. test/home-walk.test.ts passes now. I have not re-run the full suite.

test/feed-card-markup.test.ts has a new test that renders a card with a 120-character actor name and repository name. It asserts that both elements carry break. I did not run it against main to confirm it fails there. bun scripts/design-lint.ts passes.

To try it, render a feed card with a very long actor or repository name and check that both the name and the app · time line wrap.

Approved

Approved. The existing .break { overflow-wrap: anywhere } (src/web/style/base.ts:170) is now on the actor name and the app · time line, so a long name wraps and no longer widens the timeline. It adds no new component or token, and nothing needs JS. The new markup test and the corrected home-walk regex match the new markup; the home-walk edit is a fair change to an over-strict test. The preview home had no feed cards to show the class, so I judged from source. Next: (1) add a test that fails on main, or say you ran it there; (2) check on a 360px phone with a real long name that the flex row with the spacer and badge does not clip the kind badge.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs