Skip to content
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

editnew: Use raw body text instead of rendered version #1648

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

guludo
Copy link
Contributor

@guludo guludo commented Feb 19, 2024

Rendering mail content via mailcap makes sense when viewing email, but not when reusing the email for composing a new message, as it is desirable to rather edit the "original" content and, if mailcap rendering produces ANSI CSI escapes, the user would be left with a bunch of garbarge characters.

Make sure that the editnew command do not render the body when creating the envelope.

Rendering mail content via mailcap makes sense when viewing email, but
not when reusing the email for composing a new message, as it is
desirable to rather edit the "original" content and, if mailcap
rendering produces ANSI CSI escapes, the user would be left with a bunch
of garbarge characters.

Make sure that the editnew command do not render the body when creating
the envelope.
@pazz
Copy link
Owner

pazz commented Feb 20, 2024 via email

@guludo
Copy link
Contributor Author

guludo commented Feb 21, 2024 via email

@pazz
Copy link
Owner

pazz commented May 3, 2024

I think adding an option to èditnew`and perhaps a config switch for the default value of that option would make sense here, yes.

@pazz
Copy link
Owner

pazz commented Jul 17, 2024

I'm going over old/stale commits towards making a release soonish.
Would you like to repackage this with the extra option and include in a release?
I think I now tend to agree with you that this is a good idea if editnew is used for drafts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants