Page MenuHomePhabricator

Refine durable column class toggling
ClosedPublic

Authored by btrahan on May 11 2015, 8:46 PM.
Tags
None
Referenced Files
F18771700: D12795.diff
Oct 8 2025, 9:10 PM
F18747171: D12795.id.diff
Oct 3 2025, 2:58 PM
F18735437: D12795.diff
Oct 1 2025, 1:36 AM
F18657544: D12795.diff
Sep 23 2025, 2:22 AM
F18599013: D12795.id30759.diff
Sep 13 2025, 6:26 AM
F18523835: D12795.diff
Sep 6 2025, 11:22 AM
F18394504: D12795.id30760.diff
Aug 29 2025, 10:02 AM
F18384025: D12795.id30760.diff
Aug 28 2025, 11:41 PM
Subscribers

Details

Summary

don't bother with margin class if column isn't visible. Ref T8151.

Test Plan

using chrome with trackpad

  • loaded home with toggled off column - noted no margin class erroneously applied
  • toggled column off and on - correct
  • loaded home with toggled on column - correct
  • toggled column off and on - correct

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Refine durable column class toggling.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
This revision is now accepted and ready to land.May 11 2015, 8:48 PM
This revision was automatically updated to reflect the committed changes.