Skip to content

Phase 1 Refactor MImageBody to shared component#33093

Merged
ZacksBot merged 27 commits intoelement-hq:developfrom
ZacksBot:refactor/MImage-Body
Apr 16, 2026
Merged

Phase 1 Refactor MImageBody to shared component#33093
ZacksBot merged 27 commits intoelement-hq:developfrom
ZacksBot:refactor/MImage-Body

Conversation

@ZacksBot
Copy link
Copy Markdown
Member

@ZacksBot ZacksBot commented Apr 10, 2026

This is a PR for Phase 1: refactoring of MImageBody, for creating a replica of the original component in shared components

@github-actions github-actions bot added the Z-Community-PR Issue is solved by a community member's PR label Apr 10, 2026
switch (placeholder) {
case ImageBodyViewPlaceholder.BLURHASH:
if (!blurhash) {
return <InlineSpinner aria-label="Loading..." role="progressbar" />;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

missing i18n

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

);

case ImageBodyViewPlaceholder.SPINNER:
return <InlineSpinner aria-label="Loading..." role="progressbar" />;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ditto

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@ZacksBot ZacksBot added this pull request to the merge queue Apr 16, 2026
Merged via the queue into element-hq:develop with commit d4aea25 Apr 16, 2026
54 checks passed
@ZacksBot ZacksBot deleted the refactor/MImage-Body branch April 16, 2026 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning Z-Community-PR Issue is solved by a community member's PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants