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
Unknown Object (File)
Aug 20 2026, 4:13 PM
Unknown Object (File)
Aug 15 2026, 12:42 AM
Unknown Object (File)
Apr 18 2026, 12:03 PM
Unknown Object (File)
Mar 31 2026, 11:31 PM
Unknown Object (File)
Mar 14 2026, 3:57 PM
Unknown Object (File)
Mar 11 2026, 5:50 PM
Unknown Object (File)
Mar 5 2026, 7:50 AM
Unknown Object (File)
Mar 3 2026, 7:43 AM
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.