Avatars: vervang eigen initialen-cirkels door nldd-avatar (NLDD 0.8.69) #130

New issue

Closed

opened 2026-07-19 20:57:29 +00:00 by robbertbos · 1 comment

robbertbos commented

2026-07-19 20:57:29 +00:00

Owner

NLDD 0.8.69 introduceert nldd-avatar: afbeelding -> initialen -> type-icoon als fallback-keten, met automatische terugval bij een dode src.

Waggle heeft nu drie eigen implementaties van hetzelfde (initials()-util + border-radius: 50%-cirkel):

CardRow.vue (auteur-avatar, 2 plekken)
ReactorPopover.vue (reactor-lijst)
ThreadView.vue - inclusief een expliciete CSP-workaround omdat een kapotte avatar-afbeelding niet via img.style.display naar de initialen-fallback mag springen; precies dat gedrag zit nu ingebouwd in nldd-avatar.

Vervangen door nldd-avatar (type=person, name voor de initialen, src voor de MM-avatar) ruimt de drie kopieën en de CSP-workaround op.

Afhankelijk van PR #124 (bump naar ^0.8.69).

NLDD 0.8.69 introduceert `nldd-avatar`: afbeelding -> initialen -> type-icoon als fallback-keten, met automatische terugval bij een dode `src`. Waggle heeft nu drie eigen implementaties van hetzelfde (initials()-util + border-radius: 50%-cirkel): - `CardRow.vue` (auteur-avatar, 2 plekken) - `ReactorPopover.vue` (reactor-lijst) - `ThreadView.vue` - inclusief een expliciete CSP-workaround omdat een kapotte avatar-afbeelding niet via `img.style.display` naar de initialen-fallback mag springen; precies dat gedrag zit nu ingebouwd in nldd-avatar. Vervangen door `nldd-avatar` (type=person, `name` voor de initialen, `src` voor de MM-avatar) ruimt de drie kopieën en de CSP-workaround op. Afhankelijk van PR #124 (bump naar ^0.8.69).

robbertbos added the

nldd ds

label

2026-07-24 06:01:23 +00:00

robbertbos referenced this issue from a commit

2026-07-26 07:31:30 +00:00

Replace CardRow's hand-built avatar with nldd-avatar

robbertbos referenced this issue from a commit

2026-07-26 08:22:28 +00:00

Finish the avatar migration to nldd-avatar

robbertbos commented

2026-08-05 21:34:18 +00:00

Author

Owner

Done. Verified on main today: all three hand-rolled circles are gone.

CardRow.vue:347, 468 - nldd-avatar with :initials
ReactorPopover.vue:61 - nldd-avatar
ThreadView.vue:122-129 - nldd-avatar, src via the /api/mm/avatar proxy, initials as fallback

CardDetailsModal.vue:181 and MentionProfilePopover.vue:83 picked it up too, which this issue did not ask for.

Two leftovers from the migration, folded into the current NLDD pass rather than kept here: ThreadView.vue:68-71 still carries the comment describing the old @error / CSP workaround that no longer exists, and ThreadView.vue:73 has a local initials() duplicating cardViewUtils.initials.

Done. Verified on main today: all three hand-rolled circles are gone. - CardRow.vue:347, 468 - nldd-avatar with :initials - ReactorPopover.vue:61 - nldd-avatar - ThreadView.vue:122-129 - nldd-avatar, src via the /api/mm/avatar proxy, initials as fallback CardDetailsModal.vue:181 and MentionProfilePopover.vue:83 picked it up too, which this issue did not ask for. Two leftovers from the migration, folded into the current NLDD pass rather than kept here: ThreadView.vue:68-71 still carries the comment describing the old <img> @error / CSP workaround that no longer exists, and ThreadView.vue:73 has a local initials() duplicating cardViewUtils.initials.

robbertbos closed this issue

2026-08-05 21:34:23 +00:00

robbertbos referenced this issue

2026-08-05 21:35:43 +00:00

Show author avatars with nldd-avatar #167