Skip to content

Fix channels UI avatars and add peer/channel buttons#608

Merged
shocknet-justin merged 1 commit intomainfrom
cursor/channels-ui-avatars-buttons-d50e
Apr 18, 2026
Merged

Fix channels UI avatars and add peer/channel buttons#608
shocknet-justin merged 1 commit intomainfrom
cursor/channels-ui-avatars-buttons-d50e

Conversation

@shocknet-justin
Copy link
Copy Markdown
Member

Summary

  • Avatars: Channel rows were building avatar: "" and rendering a 12×12 <img>, which produced broken images and vertical alt text that broke the layout. Avatars now use a stable robohash.org URL derived from channel_id, then channel_point, then label, with 32×32 display size, object-fit: cover, and empty alt so no placeholder text appears if the image fails.
  • Labels: Long channel labels get a single-line ellipsis in the avatar row so they do not wrap awkwardly.
  • Actions: Replaced the unstyled "ADD PEER" / "ADD CHANNEL" native buttons with full-width IonButton controls (primary + outline) and scoped .Channels_open-actions styles consistent with the channels page spacing.

Testing

  • npx eslint on the touched Channels files (project-wide npm run lint has pre-existing failures).
  • npm run test.unit currently fails in unrelated specs; no new failures attributable to these changes were isolated.
Open in Web Open in Cursor 

Populate channel list avatars with stable robohash URLs from channel id
or point, use sensible image dimensions and empty alt to avoid broken-
image text layout. Ellipsize long peer labels in the flex row.

Replace plain text peer/channel triggers with Ionic buttons and scoped
styles matching the channels gradient theme.

Co-authored-by: Justin (shocknet) <shocknet-justin@users.noreply.github.com>
@shocknet-justin shocknet-justin marked this pull request as ready for review April 18, 2026 19:01
@shocknet-justin shocknet-justin merged commit 08df554 into main Apr 18, 2026
2 checks passed
@shocknet-justin shocknet-justin deleted the cursor/channels-ui-avatars-buttons-d50e branch April 18, 2026 19:02
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.

2 participants