HomePhorge

Fix a JS console warning when hovering over replies to ghosts on lines which no…

Description

Fix a JS console warning when hovering over replies to ghosts on lines which no longer exist

Summary:
Fixes T11662. In the very obscure situation described in that task, quiet a JS console warning.

The actual edit operation appears to work correctly after changes elsewhere.

There aren't really any legitimate lines for us to highlight in this case so I'm just giving up rather than trying to do something approximate.

Test Plan:

  • Wrote long.txt.
  • Created revision.
  • Added an inline near the bottom.
  • Removed most of long.txt.
  • Updated revsion.
  • Replied to the ghost inline.
  • Edited the reply to the ghost inline, worked.
  • Hovered the reply to the ghost inline: no line highlight, but no errors.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11662

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

Details

Provenance
epriestleyAuthored on May 17 2017, 8:35 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP9eb285f4aabb: Fix "left"/"right" changeset ID selection for synthetic deletions
Branches
Unknown
Tags
Unknown

Event Timeline