Skip to content

compose: Support editing a message from any message-list page, not just channel/topic/DM narrow #1518

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
chrisbobbe opened this issue May 20, 2025 · 1 comment
Labels
a-compose Compose box, autocomplete, attaching files/images beta feedback Things beta users have specifically asked for

Comments

@chrisbobbe
Copy link
Collaborator

Currently, you can edit a message from the message-list page for a channel narrow, a topic narrow, or a DM narrow. This issue is to support editing a message on any message-list page, including the combined feed, the starred-messages page, etc.

This wasn't included in #126 / PR #1498 because some refactoring is needed; the current code is written on the assumption that a given message-list page will either always have or never have a compose box, determined by the Narrow. I think the main thing built on that assumption was safe-area handling, but there may be other things.

@chrisbobbe chrisbobbe added this to the M6: Post-launch milestone May 20, 2025
@chrisbobbe chrisbobbe added the a-compose Compose box, autocomplete, attaching files/images label May 20, 2025
@gnprice gnprice added the beta feedback Things beta users have specifically asked for label May 20, 2025
@gnprice
Copy link
Member

gnprice commented May 20, 2025

Chat thread with feedback asking for this (specifically for the combined feed): #mobile > Flutter beta: cannot edit messages @ 💬

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-compose Compose box, autocomplete, attaching files/images beta feedback Things beta users have specifically asked for
Projects
Status: No status
Development

No branches or pull requests

2 participants