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)
Mon, Apr 1, 3:58 AM
Unknown Object (File)
Jan 6 2024, 8:46 AM
Unknown Object (File)
Dec 30 2023, 6:47 PM
Unknown Object (File)
Dec 27 2023, 11:01 AM
Unknown Object (File)
Dec 27 2023, 11:01 AM
Unknown Object (File)
Dec 27 2023, 11:01 AM
Unknown Object (File)
Dec 26 2023, 10:25 PM
Unknown Object (File)
Dec 26 2023, 8:23 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.