Page MenuHomePhabricator
Differential D21645

Make inline "ContentState" a client object, and track "hasSuggestion" on it
ClosedPublic
Actions

Authored by epriestley on Mar 25 2021, 8:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 15 2026, 2:59 PM
Unknown Object (File)
Apr 2 2026, 9:33 PM
Unknown Object (File)
Apr 1 2026, 6:50 PM
Unknown Object (File)
Mar 14 2026, 10:56 AM
Unknown Object (File)
Feb 11 2026, 6:43 AM
Unknown Object (File)
Jan 31 2026, 7:02 PM
Unknown Object (File)
Oct 22 2025, 9:59 AM
Unknown Object (File)
Oct 20 2025, 12:36 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
rPb964731b6a6b: Make inline "ContentState" a client object, and track "hasSuggestion" on it
Summary

Ref T13559. In an effort to ultimately fix the "quote + cancel" bug, begin formalizing content states on the client.

This creates a "ContentState" client object and moves the authoritative storage for the "hasSuggestion" property to it.

Test Plan

Created inlines, 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, 8:51 PM
Harbormaster completed remote builds in B25321: Diff 51513.Mar 25 2021, 8:52 PM
epriestley requested review of this revision.Mar 25 2021, 8:52 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2021, 4:00 PM
Closed by commit rPb964731b6a6b: Make inline "ContentState" a client object, and track "hasSuggestion" on it (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rPb964731b6a6b: Make inline "ContentState" a client object, and track "hasSuggestion" on it.

Revision Contents
Changeset List

PathSize
resources/
celerity/
1 line
webroot/
rsrc/
js/
application/
diff/
15 lines
22 lines

Diff 51539

View Options

resources/celerity/map.php

Loading...
View Options

resources/celerity/packages.php

Loading...
View Options

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

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInlineContentState.js

Loading...
Phabricator · Built by Phacility