Page MenuHomePhabricator
Differential D9545

Switch Pholio over to lighter layout.
ClosedPublic
Actions

Authored by chad on Jun 15 2014, 5:31 AM.
Tags
None
Referenced Files
F13189071: D9545.diff
Sat, May 11, 5:42 AM
Unknown Object (File)
Tue, May 7, 9:10 AM
Unknown Object (File)
Fri, May 3, 9:10 AM
Unknown Object (File)
Thu, Apr 25, 12:12 AM
Unknown Object (File)
Tue, Apr 16, 3:51 AM
Unknown Object (File)
Apr 1 2024, 10:07 PM
Unknown Object (File)
Mar 31 2024, 7:56 PM
Unknown Object (File)
Feb 14 2024, 7:58 AM
View All Files
Subscribers
epriestley
Korvin
Tokens
"Baby Tequila" token, awarded by epriestley.

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP85870e79aaf2: Switch Pholio over to lighter layout.
Summary

Gets rid of all the dark css.

Test Plan

Do it live.

Screen_Shot_2014-06-14_at_10.32.00_PM.png (1×2 px, 981 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

chad updated this revision to Diff 22876.Jun 15 2014, 5:31 AM
chad retitled this revision from to Switch Pholio over to lighter layout..
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJun 15 2014, 5:31 AM
chad edited the test plan for this revision. (Show Details)Jun 15 2014, 5:32 AM
Harbormaster completed remote builds in B1105: Diff 22876.Jun 15 2014, 5:32 AM
chad added a comment.Jun 15 2014, 5:36 AM

though if you really like the dark checker, we can still work on that

chad updated this revision to Diff 22877.Jun 15 2014, 5:45 AM
  • Make % sky
Harbormaster completed remote builds in B1106: Diff 22877.Jun 15 2014, 5:46 AM
epriestley accepted this revision.Jun 15 2014, 12:33 PM
epriestley edited edge metadata.

Let's give this a shot. The original lighter version looked iffy but this doesn't seem bad.

This revision is now accepted and ready to land.Jun 15 2014, 12:33 PM
chad closed this revision.Jun 15 2014, 2:44 PM
chad updated this revision to Diff 22881.

Closed by commit rP85870e79aaf2 (authored by @chad).

epriestley added a comment.Jun 15 2014, 3:02 PM

I really like the light backgrounds after playing with them for a bit.

epriestley awarded a token.Jun 15 2014, 3:02 PM

Revision Contents
Changeset List

PathSize
resources/
celerity/
53 lines
src/
applications/
pholio/
controller/
4 lines
view/
13 lines
webroot/
rsrc/
css/
application/
pholio/
60 lines
48 lines
image/

Diff 22881

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/pholio/controller/PholioMockViewController.php

Loading...
View Options

src/applications/pholio/view/PholioMockThumbGridView.php

Loading...
View Options

webroot/rsrc/css/application/pholio/pholio-inline-comments.css

Loading...
View Options

webroot/rsrc/css/application/pholio/pholio.css

Loading...
View Options

webroot/rsrc/image/checker_lighter.png

Loading...
Phabricator · Built by Phacility