Skip to content

Conversation

@dev-rb
Copy link
Contributor

@dev-rb dev-rb commented Jan 21, 2026

This PR adds the ability to schedule a message and modify or remove the scheduled time.

In addition:

  • Updates the icon for time inputs to use a Phosphor icon and makes the icon react to theme changes to allow custom colors that respect light/dark themes
  • Adds mutations for saving and deleting drafts
  • Adds an indicator to BaseInput and Compose to indicate when an email is being saved and has been saved
  • Fixes an issue where the recipient options in EmailContext were returned as different entity "kinds"
  • Removes the bottom input for email threads and makes it so the reply box is the new default and always open for the last message. This makes it slightly more clear what is being modified
  • Adds a generic DateSelector component
  • Places this feature behind a ff
Screen.Recording.2026-01-23.at.3.07.41.PM.mov

@github-actions
Copy link

github-actions bot commented Jan 21, 2026

@dev-rb dev-rb marked this pull request as ready for review January 23, 2026 19:06
@dev-rb dev-rb requested a review from a team as a code owner January 23, 2026 19:06
@claude
Copy link
Contributor

claude bot commented Jan 23, 2026


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.


@evanhutnik
Copy link
Contributor

Personally, I'm not a fan of removing the compose box from the bottom of the thread view. It makes it so that you can't scroll up and read emails in the thread while you are writing a message.

Also, running locally, when I open email threads there is no reply box at the bottom like you mentioned in the PR description

@dev-rb
Copy link
Contributor Author

dev-rb commented Feb 2, 2026

Personally, I'm not a fan of removing the compose box from the bottom of the thread view. It makes it so that you can't scroll up and read emails in the thread while you are writing a message.

Also, running locally, when I open email threads there is no reply box at the bottom like you mentioned in the PR description

Fair enough, I think I'll add the bottom compose box back and leave the decision to remove it to another time if it comes up as an issue.

@evanhutnik
Copy link
Contributor

A common use case for scheduled send is to send something the next day, at the start of the business day (i.e. 9AM). I can't figure out how to set the date AND time I want a message to be sent with the current UI.

@dev-rb
Copy link
Contributor Author

dev-rb commented Feb 2, 2026

A common use case for scheduled send is to send something the next day, at the start of the business day (i.e. 9AM). I can't figure out how to set the date AND time I want a message to be sent with the current UI.

I removed the time selector in one of my recent commits because it doesn't fit the rest of the theme of the app since it's the native selector and it isn't able to be styled. So maybe we can just ff this feature until time selection is added

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.

5 participants