Page MenuHomePhabricator

Mostly move inline editing to DiffInline
ClosedPublic

Authored by epriestley on May 15 2017, 7:53 PM.
Tags
None
Referenced Files
F18802996: D17887.id43026.diff
Fri, Oct 17, 9:11 PM
F18683667: D17887.diff
Fri, Sep 26, 8:02 AM
F18644445: D17887.diff
Sep 19 2025, 5:19 AM
F18587679: D17887.diff
Sep 11 2025, 10:04 PM
F18508789: D17887.id.diff
Sep 5 2025, 3:01 AM
F18501025: D17887.diff
Sep 4 2025, 9:29 PM
F18217635: D17887.id.diff
Aug 19 2025, 9:07 AM
F18207132: D17887.diff
Aug 18 2025, 4:47 PM
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.