Page MenuHomePhabricator

Colorized html inline diffs for Diffusion
Open, Needs TriagePublic

Description

T10694 brought in a nice inline html diff rendering for differential emails. It would be useful if that was brought over to diffusion in two places:

  1. Inline comments in audit
  2. Commit message emails

Reasons for wanting this:

  • It is consistent. I was confused that the red/green html diff is only available in one place and not another
  • We review code after it is committed, so being able to quickly read an email diff and see the changes is really helpful. Anything that makes reading those emails easier saves a lot of time.
  • It is pretty :-)

Event Timeline

chad renamed this task from Colourised html inline diffs for diffusion to Colorized html inline diffs for Diffusion.Jun 9 2016, 3:17 AM
epriestley edited projects, added Diffusion (v3); removed Diffusion.