Maybe I'm the only person on earth who still prefers plain text emails most of the time. ;) But, it'd be relatively easy to add a config knob to let site admins decide if there should be an option on the forward form to select the email format you want to send someone. It'd all default to HTML, but if you go out of your way to allow it, you can give your site users the freedom to send what they prefer.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | 2969352-10.plain-text-needs-helper-class.patch | 412 bytes | dww |
| #4 | 2969352-interdiff.2_3.txt | 498 bytes | dww |
| #4 | 2969352-3.allow-plain-text.patch | 6.19 KB | dww |
| #2 | 2969352-2.allow-plain-text.patch | 5.12 KB | dww |
Comments
Comment #2
dwwComment #3
dwwComment #4
dwwThis adds
forward_update_8200()to set the default for the new config knob (since this is a stable module and all that). ;)p.s. Interdiff seems to be confused. Here's a raw diff of the two patch files.
Comment #5
dww(Sorry, cut+paste error).
Comment #7
john.oltman commentedComment #8
dwwCool, thanks! :) I thought I might have to put up a little fight around this one. ;)
Cheers,
-Derek
Comment #9
john.oltman commentedNo problem sir! I'm happy to add features for people willing to spend time testing and patching, especially ones that are relatively benign to existing installs.
Comment #10
dwwWhoops! This is what happens when I'm developing locally and using devel to handle mail. ;) Now that I deployed this to a real server to test it, I discovered this patch was missing a use statement it needs. Sorry.
Comment #12
john.oltman commentedComment #13
dwwSweet, thanks! Interesting that d.o doesn't show the commits to 8.x-3.x in this issue. Wonder what's up with that. Anyway, all looks good. Neither commit is in 8.x-2.x now, and both are in 8.x-3.x.
Yay,
-Derek