Skip to content

feat(email): toggle quoted text in reply#763

Merged
dev-rb merged 14 commits intomainfrom
rahul/comms-85-move-append-reply-structure-button
Dec 29, 2025
Merged

feat(email): toggle quoted text in reply#763
dev-rb merged 14 commits intomainfrom
rahul/comms-85-move-append-reply-structure-button

Conversation

@dev-rb
Copy link
Copy Markdown
Contributor

@dev-rb dev-rb commented Dec 26, 2025

Summary

Screenshots, GIFs, and Videos

Quoted text off:
Screenshot 2025-12-26 at 12 11 23 PM

Quoted text on:
Screenshot 2025-12-26 at 12 11 33 PM

@linear
Copy link
Copy Markdown

linear bot commented Dec 26, 2025

@dev-rb dev-rb changed the title feat(email): toggle quoted text feat(email): toggle quoted text in reply Dec 26, 2025
@dev-rb dev-rb marked this pull request as ready for review December 26, 2025 20:58
@dev-rb dev-rb requested a review from a team as a code owner December 26, 2025 20:58
@peterchinman
Copy link
Copy Markdown
Contributor

UI thoughts:

  1. No text, just the quote icon with a tool tip.
  2. Move it to the left of the trash icon (it feels janky when it switches positions).

@peterchinman
Copy link
Copy Markdown
Contributor

Functional thoughts:

This doesn't survive serialization to html and back, which is how drafts work right now. If you press this button to append quoted reply, let the draft save, and reload—you are no longer able to remove quoted section with the button.

@dev-rb dev-rb merged commit 47d9f61 into main Dec 29, 2025
19 checks passed
@dev-rb dev-rb deleted the rahul/comms-85-move-append-reply-structure-button branch December 29, 2025 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants