Homec4science

Partially revert D14511 to fix "INLINE COMMENTS" in mail

Authored by epriestley <git@epriestley.com> on Nov 28 2015, 21:45.

Description

Partially revert D14511 to fix "INLINE COMMENTS" in mail

Summary:
Ref T9845. In Differential, this is not a remarkup block -- it's a mail section. addTextSection() has special magic behavior when handed a prebuilt section since D9375.

Swapping to addRemarkupSection() causes the error in T9845 and renders nothing in the comment section.

Even if it were a block of text, it would not be appropriate to add it as remarkup. This would incorrectly render comments in files like __init__.py, which are common on Python (the filename would render as "__init__.py"). Okay that's a bad example since it works fine but, uh, a file named T123 would be no good or whatever.

I'll realign T9845 to clean this up and fix it more durably.

Test Plan: Sent myself some mail with inline comments, saw them in the mail.

Reviewers: joshuaspence, chad

Reviewed By: chad

Maniphest Tasks: T9845

Differential Revision: https://secure.phabricator.com/D14589

Details

Committed
epriestley <git@epriestley.com>Nov 28 2015, 22:40
Pushed
aubortJan 31 2017, 17:16
Parents
rPH5eada3d89c8a: Add Profile Images to PhameBlog
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH0d01dab5a375: Partially revert D14511 to fix "INLINE COMMENTS" in mail (authored by epriestley <git@epriestley.com>).Nov 28 2015, 22:40