Page MenuHomePhabricator
Differential D21139

Remove the "Filetree" preference, so it's on for everyone
AbandonedPublic
Actions

Authored by epriestley on Apr 18 2020, 4:48 PM.
Tags
None
Referenced Files
F19154197: D21139.id50336.diff
Thu, Dec 11, 1:49 PM
F19152728: D21139.id.diff
Thu, Dec 11, 4:46 AM
F19151003: D21139.id50336.diff
Wed, Dec 10, 9:52 PM
F19078407: D21139.id.diff
Mon, Dec 1, 6:07 PM
F19072145: D21139.diff
Sun, Nov 30, 9:32 PM
F19043632: D21139.diff
Wed, Nov 26, 11:40 PM
F18985434: D21139.id50336.diff
Nov 17 2025, 1:02 PM
F18843941: D21139.id.diff
Oct 29 2025, 2:06 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13516: Remove the "filetree" setting (so it's always enabled for everyone)
Summary

Ref T13516. This is an ancient setting which was motivated by performacne, but is confusing and no longer really relevant. Get rid of it.

Test Plan

Grepped for affected symbols, pressed "F" to toggle the filetree.

Diff Detail

Repository
rP Phabricator
Branch
sticky3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24154
Build 33263: Run Core Tests
Build 33262: arc lint + arc unit

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
8 lines
diffusion/
controller/
6 lines
settings/
setting/

Diff 50336

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
View Options

src/applications/settings/setting/PhabricatorShowFiletreeSetting.php

Loading...
Phabricator · Built by Phacility