Page MenuHomePhabricator

Clean up more Quicksand
ClosedPublic

Authored by chad on Oct 1 2016, 5:23 AM.
Tags
None
Referenced Files
F19829343: D16642.id.diff
Tue, Mar 10, 4:13 AM
F19651356: D16642.id40068.diff
Feb 6 2026, 11:23 AM
F19531273: D16642.diff
Jan 20 2026, 3:28 AM
F19531269: D16642.diff
Jan 20 2026, 3:26 AM
F19527672: D16642.diff
Jan 19 2026, 3:24 AM
F19504547: D16642.id40066.diff
Jan 9 2026, 5:19 AM
F19468881: D16642.id40068.diff
Jan 4 2026, 12:11 PM
F19101483: D16642.diff
Dec 5 2025, 2:52 AM
Subscribers

Details

Summary

Creates a background that renders inside the Quicksand frame, through sorcery.

Test Plan

Turn on Quicksand, visit lots of pages. See correct background colors. This probably blows something up I'm not testing.

Diff Detail

Repository
rP Phabricator
Branch
quicksand-2 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13954
Build 18075: Run Core Tests
Build 18074: arc lint + arc unit

Event Timeline

chad retitled this revision from to Clean up more Quicksand.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
webroot/rsrc/css/application/base/standard-page-view.css
20–31

whatcouldgowrong

When that background stuff breaks something I guess we can special case <body /> classes.

epriestley edited edge metadata.
This revision is now accepted and ready to land.Oct 1 2016, 6:20 PM

it seemed pretty resilient, oddly, but still feels like it should break because CSS?

This revision was automatically updated to reflect the committed changes.