Page MenuHomePhabricator

Remove "Moved Document from ..." notice in Phriction
ClosedPublic

Authored by epriestley on Sep 12 2019, 4:04 AM.
Tags
None
Referenced Files
F14085933: D20800.id49604.diff
Sat, Nov 23, 1:31 PM
F14085774: D20800.id49596.diff
Sat, Nov 23, 1:02 PM
Unknown Object (File)
Thu, Nov 21, 2:53 PM
Unknown Object (File)
Thu, Nov 21, 2:31 AM
Unknown Object (File)
Sat, Nov 16, 8:30 PM
Unknown Object (File)
Oct 24 2024, 9:29 AM
Unknown Object (File)
Oct 14 2024, 12:48 AM
Unknown Object (File)
Oct 2 2024, 1:04 AM
Subscribers
None
Tokens
"Like" token, awarded by leoluk.

Details

Summary

Ref T13410. See PHI1431. Currently, when you move a document in Phriction, the target shows a "This document was moved from ..." banner until it is edited.

This banner isn't particularly useful, and it's distracting and it isn't obvious how to dismiss it, and making a trivial edit to dismiss it is awkward.

This information is also already available in the transaction log.

Just remove this banner since it doesn't really serve any clear purpose.

Test Plan
  • Moved a page in Phriction, then loaded the destination page. Before change: header banner. After change: nothing.
  • Viewed a normal (non-moved) page, saw normal behavior.
  • Reviewed transactions, saw "Moved from ..." in the timeline.

Diff Detail

Repository
rP Phabricator
Branch
phriction1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23421
Build 32176: Run Core Tests
Build 32175: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Sep 12 2019, 4:32 PM
This revision was automatically updated to reflect the committed changes.