HomePhorge

When replying to a ghost comment, attach the reply to the same place

Description

When replying to a ghost comment, attach the reply to the same place

Summary:
Fixes T10562. I left this behavior sort of ambiguous in the original implementation because I didn't anticipate or stumble across this situation.

It's easy to fix: when you reply to a ghost, just put the reply in the exact same place as the ghost (even if it's a different diff), so they always move/ghost/port/thread together.

Test Plan:
See T10562 for reproduction steps and a "before" picture. Here's the after picture:

{F1168983}

The two comments at the bottom are pre-fix, and exhibit the bug. The comment at the top is post-fix, and appears adjacent to the original correctly.

Reviewers: chad

Reviewed By: chad

Subscribers: eadler

Maniphest Tasks: T10562

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

Details

Provenance
epriestleyAuthored on Mar 10 2016, 4:05 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP7b8da9991429: Move DifferentialRevisionViewController to newPage()
Branches
Unknown
Tags
Unknown

Event Timeline