Page MenuHomePhabricator

When deleting inline comments, offer "undo" instead of prompting
ClosedPublic

Authored by epriestley on Mar 9 2015, 10:59 PM.
Tags
None
Referenced Files
F15437610: D12032.id28963.diff
Tue, Mar 25, 8:40 PM
F15437609: D12032.id.diff
Tue, Mar 25, 8:40 PM
F15432709: D12032.id28963.diff
Mon, Mar 24, 7:30 PM
F15429807: D12032.diff
Mon, Mar 24, 4:23 AM
F15390331: D12032.id28962.diff
Sat, Mar 15, 6:07 AM
F15367457: D12032.diff
Tue, Mar 11, 6:51 PM
Unknown Object (File)
Feb 23 2025, 5:25 AM
Unknown Object (File)
Feb 19 2025, 9:03 AM
Subscribers
Tokens
"Haypence" token, awarded by btrahan.

Details

Summary

Ref T2009. Ref T1460.

Fixes T2618. When users hit "Delete" on inline comments, delete immediately and offer them "Undo". If they delete indirectly (e.g., by clicking "Delete" from the preview at the bottom of the page), we still prompt them, because the "Undo" action either won't be available or may not be easy to find. This is a "refdelete".

Fixes T6464. This was just a mess. Make it not as much of a mess. It should work now. Pretty sure.

Fixes T4999. We did not refresh these links often enough to find targets for them, so they could race with content. Reevaluate them after loading new changes.

Test Plan
  • Deleted and undid deletion of inlines from main view and preview.
  • Clicked "View" on inlines.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable