Page MenuHomePhabricator

Add a preference to disable ghost comments
ClosedPublic

Authored by epriestley on May 4 2015, 6:55 PM.
Tags
None
Referenced Files
F15420869: D12704.id30502.diff
Fri, Mar 21, 6:26 PM
F15388087: D12704.id30504.diff
Sat, Mar 15, 3:18 AM
F15301371: D12704.id.diff
Wed, Mar 5, 5:45 PM
F15285275: D12704.diff
Tue, Mar 4, 12:35 PM
Unknown Object (File)
Feb 21 2025, 10:16 PM
Unknown Object (File)
Feb 16 2025, 4:53 PM
Unknown Object (File)
Feb 9 2025, 6:29 PM
Unknown Object (File)
Feb 8 2025, 4:17 AM

Details

Summary

Ref T7447. At least some users dislike this feature so strongly that they'd prefer not to have it at all.

Test Plan

Viewed ghosts; toggled preference, no more ghosts.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add a preference to disable ghost comments.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

We could add an "Only if not Done" option easily, but bringing "Done" comments forward is so valuable for me that it's hard for me to imagine that this is really a useful setting.

I could perhaps imagine an "On revisions I am the author of, only if not Done" setting, but this seems too complicated and confusing.

chad edited edge metadata.
This revision is now accepted and ready to land.May 4 2015, 7:01 PM
This revision was automatically updated to reflect the committed changes.