Skip to content

Conversation

chrisbobbe
Copy link
Collaborator

Related: #1913

@chrisbobbe chrisbobbe added the maintainer review PR ready for review by Zulip maintainers label Oct 16, 2025
Copy link
Member

@rajveermalviya rajveermalviya left a comment

Choose a reason for hiding this comment

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

Thanks @chrisbobbe! LGTM, moving over to Greg's review.

@rajveermalviya rajveermalviya added integration review Added by maintainers when PR may be ready for integration and removed maintainer review PR ready for review by Zulip maintainers labels Oct 17, 2025
Copy link
Member

@gnprice gnprice left a comment

Choose a reason for hiding this comment

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

Sure, sounds good. One nit below, then please merge at will.


class MessageImage extends StatelessWidget {
const MessageImage({super.key, required this.node});
class MessageImagePreview extends StatelessWidget {
Copy link
Member

Choose a reason for hiding this comment

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

nit: git grep -w MessageImage finds one straggler reference in a comment

The linked discussion is about adding documentation; we can link to
that documentation once it's added.
…age"

The API doc is being updated to distinguish the current "image
preview" content node from a new one, "image element presented in
Markdown syntax", which I think we'll just call "inline image". See
PR in progress: zulip/zulip#36226

Related: zulip#1913
@chrisbobbe chrisbobbe force-pushed the pr-rename-image-to-image-preview branch from c63d768 to 51dbd2c Compare October 20, 2025 21:07
@chrisbobbe chrisbobbe merged commit 51dbd2c into zulip:main Oct 20, 2025
1 check passed
@chrisbobbe
Copy link
Collaborator Author

Thanks! Done.

@chrisbobbe chrisbobbe deleted the pr-rename-image-to-image-preview branch October 20, 2025 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration review Added by maintainers when PR may be ready for integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants