Page MenuHomePhabricator

Mostly move inline editing to DiffInline
ClosedPublic

Authored by epriestley on May 15 2017, 7:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 15 2026, 8:47 PM
Unknown Object (File)
Feb 6 2026, 5:45 AM
Unknown Object (File)
Feb 1 2026, 7:25 PM
Unknown Object (File)
Feb 1 2026, 7:25 PM
Unknown Object (File)
Feb 1 2026, 3:40 AM
Unknown Object (File)
Jan 9 2026, 8:16 AM
Unknown Object (File)
Jan 8 2026, 4:47 AM
Unknown Object (File)
Oct 30 2025, 10:49 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.