Page MenuHomePhabricator

Make the filetree view width sticky across show/hide and reload
ClosedPublic

Authored by epriestley on Feb 22 2018, 9:44 PM.
Tags
None
Referenced Files
F19906402: D19129.id45835.diff
Tue, Mar 31, 11:31 PM
F19865937: D19129.diff
Mar 14 2026, 3:57 PM
F19839606: D19129.diff
Mar 11 2026, 5:50 PM
F19816695: D19129.diff
Mar 5 2026, 7:50 AM
F19808758: D19129.diff
Mar 3 2026, 7:43 AM
F19282480: D19129.id45834.diff
Dec 23 2025, 4:16 AM
F19209636: D19129.id45834.diff
Dec 17 2025, 12:12 AM
F18905990: D19129.id.diff
Nov 8 2025, 4:27 PM
Subscribers
None

Details

Summary

Ref T13090. The default width changed recently to become much wider, but the behavior on this control isn't great. Instead:

  • Pick a default width somewhere between the two.
  • Make the width sticky across show/hide (pressing "f" twice remembers your width instead of resetting it).
  • Make the width sticky across reloads (dragging the bar, then reloading the page keeps the bar in the same place).
Test Plan
  • Without settings, loaded page: got medium-width bar.
  • Dragged bar wide/narrow, toggled on/off with "f", got persistent width.
  • Dragged bar wide/narrow, reloaded page, got persistent width.
  • Dragged bar wide/narrow, toggled it off, reloaded page, toggled it on, got persistent width.

Diff Detail

Repository
rP Phabricator
Branch
treedrag1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19643
Build 26595: Run Core Tests
Build 26594: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 22 2018, 9:47 PM
This revision was automatically updated to reflect the committed changes.