Page MenuHomePhabricator
Differential D12727

Quicksand + Pholio - make mock view work
ClosedPublic
Actions

Authored by btrahan on May 5 2015, 10:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 3, 10:50 PM
Unknown Object (File)
Fri, Jan 24, 1:00 PM
Unknown Object (File)
Fri, Jan 24, 4:04 AM
Unknown Object (File)
Fri, Jan 24, 4:04 AM
Unknown Object (File)
Fri, Jan 24, 4:04 AM
Unknown Object (File)
Fri, Jan 24, 4:04 AM
Unknown Object (File)
Fri, Jan 24, 4:04 AM
Unknown Object (File)
Tue, Jan 21, 11:03 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T8036: Pholio images don't switch in Quicksand
Commits
Restricted Diffusion Commit
rP179948e23273: Quicksand + Pholio - make mock view work
Summary

Ref T8036. This was a lot of JS to fiddle with.

Test Plan

viewed a mock with durable column open, clicked back to pholio home, then clicked back to mock and was able to switch images. clicked to profile and then hit browser back and was still able to switch images.

Diff Detail

Repository
rP Phabricator
Branch
T8036
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5784
Build 5803: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 30578.May 5 2015, 10:41 PM
btrahan added a task: T8036: Pholio images don't switch in Quicksand.
btrahan retitled this revision from to Quicksand + Pholio - make mock view work.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMay 5 2015, 10:41 PM
epriestley accepted this revision.May 5 2015, 10:52 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 5 2015, 10:52 PM
btrahan updated this revision to Diff 30587.May 5 2015, 11:18 PM
btrahan edited edge metadata.

celery

Closed by commit rP179948e23273: Quicksand + Pholio - make mock view work (authored by btrahan, committed by btrahan). · Explain WhyMay 5 2015, 11:19 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
32 lines
src/
applications/
pholio/
controller/
6 lines
view/
74 lines
webroot/
rsrc/
js/
application/
pholio/
633 lines

Diff 30578

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/pholio/controller/PholioMockViewController.php

Loading...
View Options

src/applications/pholio/view/PholioMockImagesView.php

Loading...
View Options

webroot/rsrc/js/application/pholio/behavior-pholio-mock-view.js

Loading...
Phabricator · Built by Phacility