1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-19 20:10:55 +01:00
phorge-phorge/webroot/rsrc/js/application
epriestley de7aa2186c Resolve great internal confusion about left vs right inline comments
Summary:
This code was just all kinds of wrong, but got all the common cases anyone cares
about correct.

  - In edit-inline-comments.js, if isOnRight() is true, use data.right, not
data.left (derp).
  - Set data.left correctly, not to the same value as data.right (derp derp).
  - Set "isNewFile" based on $is_new, not $on_right (derp derp derp).

Test Plan:
 - Added JS debugging code to print "OLD" vs "NEW" and "LEFT" vs "RIGHT".
Clicked the left and right sides of diff-vs-base and diff-vs-diff diffs,
verified output was accurate in all cases.
 - Added comments to the left-display-side of a diff-of-diffs, saved them, they
showed up where I put them.

Reviewers: btrahan, vrana

Reviewed By: btrahan

CC: aran, epriestley

Maniphest Tasks: T543

Differential Revision: https://secure.phabricator.com/D1567
2012-02-03 15:26:47 -08:00
..
core Fill <a href> in PhabricatorMenuItem 2012-01-17 10:49:39 -08:00
countdown Remove usage of JX.defer in favor of setTimeout 2011-08-10 18:09:59 -07:00
differential Resolve great internal confusion about left vs right inline comments 2012-02-03 15:26:47 -08:00
diffusion Remove usage of JX.defer in favor of setTimeout 2011-08-10 18:09:59 -07:00
herald Enable Phabricator admin to change the owner of a herald rule 2012-01-18 11:59:35 -08:00
maniphest Add a description preview to maniphest create / edit panel 2012-01-25 11:28:08 -08:00
owners Bring Javelin into Phabricator via git submodule, not copy-and-paste 2011-05-08 13:20:10 -07:00
phriction Add a document preview to Phriction 2011-07-16 18:48:27 -07:00
projects Remove "Former" project members 2011-12-16 17:46:02 -08:00
repository Tie all the pieces for symbol cross-references together 2011-10-09 17:58:17 -07:00
uiexample Examples using JX.View 2011-11-06 15:17:00 -08:00