Page MenuHomePhabricator

Refine durable column class toggling
ClosedPublic

Authored by btrahan on May 11 2015, 8:46 PM.
Tags
None
Referenced Files
F18747171: D12795.id.diff
Fri, Oct 3, 2:58 PM
F18735437: D12795.diff
Wed, Oct 1, 1:36 AM
F18657544: D12795.diff
Tue, Sep 23, 2:22 AM
F18599013: D12795.id30759.diff
Sat, Sep 13, 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
F18353971: D12795.id30759.diff
Aug 27 2025, 1:38 AM
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.