Page MenuHomePhabricator

Mostly move inline editing to DiffInline
ClosedPublic

Authored by epriestley on May 15 2017, 7:53 PM.
Tags
None
Referenced Files
F13254411: D17887.diff
Sat, May 25, 3:24 AM
F13236903: D17887.diff
Tue, May 21, 11:26 AM
F13233679: D17887.diff
Tue, May 21, 2:29 AM
F13219063: D17887.diff
Sat, May 18, 2:56 PM
F13211844: D17887.diff
Fri, May 17, 6:09 AM
F13196926: D17887.diff
Sun, May 12, 11:41 PM
Unknown Object (File)
May 3 2024, 7:51 AM
Unknown Object (File)
May 2 2024, 10:02 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.