Page MenuHomePhabricator

Mostly move inline editing to DiffInline
ClosedPublic

Authored by epriestley on May 15 2017, 7:53 PM.
Tags
None
Referenced Files
F15916307: D17887.diff
Sun, Jun 1, 7:56 PM
Unknown Object (File)
Wed, May 7, 11:32 PM
Unknown Object (File)
Apr 27 2025, 2:08 AM
Unknown Object (File)
Apr 7 2025, 6:31 AM
Unknown Object (File)
Apr 5 2025, 9:17 AM
Unknown Object (File)
Apr 4 2025, 5:14 PM
Unknown Object (File)
Mar 18 2025, 4:32 PM
Unknown Object (File)
Feb 28 2025, 4:58 AM
Subscribers
None

Details

Summary

Ref T12616. This doesn't pull over everything (some UI feedback didn't make it yet, and you can't cancel + undo cancelling edits yet) but editing comments technically works.

This is a little shaky, but feels less shaky than every other approach I've tried, so I think I'm finally on a reasonable track here.

Test Plan

Edited some inline comments.

Diff Detail

Repository
rP Phabricator
Branch
inline8
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16953
Build 22657: Run Core Tests
Build 22656: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.May 15 2017, 10:52 PM
This revision was automatically updated to reflect the committed changes.