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
F15180322: D21645.diff
Wed, Feb 19, 7:08 PM
Unknown Object (File)
Fri, Feb 14, 1:03 AM
Unknown Object (File)
Mon, Feb 10, 3:04 AM
Unknown Object (File)
Sat, Feb 8, 12:38 AM
Unknown Object (File)
Mon, Feb 3, 2:12 AM
Unknown Object (File)
Mon, Jan 27, 1:48 PM
Unknown Object (File)
Wed, Jan 22, 5:18 AM
Unknown Object (File)
Wed, Jan 22, 1:58 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
Branch
istate2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25321
Build 34957: Run Core Tests
Build 34956: arc lint + arc unit

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 51513

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