Page MenuHomePhabricator
Differential D21651

On inline comments, track an explicit "committed" content state
ClosedPublic
Actions

Authored by epriestley on Mar 25 2021, 9:05 PM.
Tags
None
Referenced Files
F18830885: D21651.diff
Sat, Oct 25, 9:28 AM
F18783717: D21651.id51547.diff
Mon, Oct 13, 9:04 AM
F18734413: D21651.id.diff
Sep 30 2025, 10:53 PM
F18657039: D21651.id.diff
Sep 23 2025, 12:18 AM
F18600748: D21651.diff
Sep 13 2025, 10:30 AM
F18599971: D21651.id.diff
Sep 13 2025, 8:43 AM
F18594023: D21651.diff
Sep 12 2025, 4:45 PM
F18571890: D21651.id51547.diff
Sep 10 2025, 5:28 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13559: Include "Moved/Copied" gutter in 1-up diffs, and add aural cues, coverage, and fix inline counting
Commits
rP5efe7fb4c181: On inline comments, track an explicit "committed" content state
Summary

Ref T13559. To allow the client to make correct decisions about what buttons mean, track an explicit "Committed" content state.

This is the last version of the comment that has been saved on the server, and does not exist if the comment has never been saved.

Test Plan

Created comments, etc. See followups.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 25 2021, 9:05 PM
Harbormaster completed remote builds in B25328: Diff 51520.Mar 25 2021, 9:06 PM
epriestley requested review of this revision.Mar 25 2021, 9:06 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2021, 4:01 PM
Closed by commit rP5efe7fb4c181: On inline comments, track an explicit "committed" content state (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP5efe7fb4c181: On inline comments, track an explicit "committed" content state.

Revision Contents
Changeset List

PathSize
resources/
celerity/
webroot/
rsrc/
externals/
javelin/
lib/
4 lines
js/
application/
diff/
32 lines

Diff 51547

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/externals/javelin/lib/Resource.js

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInline.js

Loading...
Phabricator · Built by Phacility