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
F13061436: D19129.diff
Fri, Apr 19, 7:26 PM
Unknown Object (File)
Thu, Apr 11, 9:30 AM
Unknown Object (File)
Wed, Apr 3, 12:01 AM
Unknown Object (File)
Sat, Mar 23, 5:07 AM
Unknown Object (File)
Mar 12 2024, 5:06 PM
Unknown Object (File)
Mar 5 2024, 8:04 AM
Unknown Object (File)
Feb 18 2024, 8:40 AM
Unknown Object (File)
Jan 27 2024, 11:25 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.