Page MenuHomePhabricator

Quicksand and Pholio - make mock edit work
ClosedPublic

Authored by btrahan on May 6 2015, 7:12 PM.
Tags
None
Referenced Files
F19512703: D12739.id30608.diff
Tue, Jan 13, 1:26 AM
F19104841: D12739.id.diff
Dec 5 2025, 3:38 PM
F19094912: D12739.diff
Dec 4 2025, 3:23 AM
F18831842: D12739.id.diff
Oct 25 2025, 3:54 PM
F18826282: D12739.diff
Oct 24 2025, 3:20 AM
F18806858: D12739.id30646.diff
Oct 18 2025, 9:52 PM
F18761750: D12739.id.diff
Oct 6 2025, 4:06 PM
F18736745: D12739.diff
Oct 1 2025, 7:38 AM
Subscribers

Details

Summary

Fixes T8036. In addition to making the mock edit work, this tightens quicksand code such that the correct page id is returned even if start() has not been called yet. It also tightens mock view where some functions should respect statics.enabled a bit more.

Test Plan

clicked edit mock, mock crumb, edit mock, mock crum, edit mock, made edits and they worked! clicked edit mock, mock crumb, edit mock, mock crumb, edit mock, profile icon, hit browser back to edit mock, made edits and they worked!

also observed mock view page not occasionally wigging out from image_onload race not having statics.enabled respect during the above

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Quicksand and Pholio - make mock edit work.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
webroot/rsrc/externals/javelin/lib/Quicksand.js
75

i'll change this to a === null check

use === null check instead
redo celery

epriestley edited edge metadata.
This revision is now accepted and ready to land.May 7 2015, 1:31 AM
This revision was automatically updated to reflect the committed changes.