Bug 1727493 Comment 77 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Henry Wilkes [:henry] from comment #71)
> there are only really 5 states:
> 
> 1. Always only send plain text.
> 2. Always only send html.
> 3. Always send both html and plain text.
> 4. Send plain text if the message is plain, otherwise send both html and plain text.
> 5. Send plain text if the message is plain, otherwise send only html.

Note, if we're ditching state 5, then the remaining options align with the 4 menu items in the compose window's "Options: Delivery Format" menu, with state 4 aligning with "Auto-detect". So we could use this preference to simply select one of the 4 by default when loading the compose window. We could drop a bunch of C++ code as well.

So it might make sense to keep all options 1 to 4, and make the wording in the preferences and the menu align.

In terms of migration, I think we could safely migrate state 5 to state 2.
(In reply to Henry Wilkes [:henry] from comment #71)
> there are only really 5 states:
> 
> 1. Always only send plain text.
> 2. Always only send html.
> 3. Always send both html and plain text.
> 4. Send plain text if the message is plain, otherwise send both html and plain text.
> 5. Send plain text if the message is plain, otherwise send only html.

Note, if we're ditching state 5, then the remaining options align with the 4 menu items in the compose window's "Options: Delivery Format" menu, with state 4 aligning with "Auto-detect". So we could use this preference to simply select one of the 4 by default when loading the compose window. We could drop a bunch of C++ code as well.

So it might make sense to keep all options 1 to 4, and make the wording in the preferences and the menu align.

In terms of migration, I think we could safely migrate state 5 to state 2 or state 4. I think probably state 4 since it is an odd configuration, and so switching to the default would seem best.
(In reply to Henry Wilkes [:henry] from comment #71)
> there are only really 5 states:
> 
> 1. Always only send plain text.
> 2. Always only send html.
> 3. Always send both html and plain text.
> 4. Send plain text if the message is plain, otherwise send both html and plain text.
> 5. Send plain text if the message is plain, otherwise send only html.

Note, if we're ditching state 5, then the remaining options align with the 4 menu items in the compose window's "Options: Delivery Format" menu, with state 4 aligning with "Auto-detect". So we could use this preference to simply select one of the 4 by default when loading the compose window. We could drop a bunch of C++ code as well.

So it might make sense to keep all options 1 to 4, and make the wording in the preferences and the menu align.

In terms of migration, I think we could safely migrate state 5 to state 2 or state 4. Since state 5 is an odd configuration, I think probably translate it to the default state 4.

Back to Bug 1727493 Comment 77