Page MenuHomePhabricator
Differential D14421

Clean up Oblivious skin
ClosedPublic
Actions

Authored by chad on Nov 6 2015, 2:58 AM.
Tags
None
Referenced Files
F15588559: D14421.id.diff
Fri, May 9, 8:16 AM
F15587444: D14421.id34841.diff
Thu, May 8, 11:03 PM
F15587443: D14421.id34842.diff
Thu, May 8, 11:03 PM
F15573591: D14421.id34841.diff
Mon, May 5, 9:18 PM
F15561086: D14421.id34841.diff
Tue, Apr 29, 6:04 PM
F15560190: D14421.id34842.diff
Tue, Apr 29, 1:23 PM
F15557232: D14421.id.diff
Mon, Apr 28, 8:36 PM
F15550393: D14421.diff
Sun, Apr 27, 11:35 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9546: Phame "Oblivious" skin a few rough edges
Commits
Restricted Diffusion Commit
rP3a90cb56b475: Clean up Oblivious skin
Summary

Adds Remarkup rules and CSS, cleans up some spacing a color. Ref T9546

Test Plan

Review a blog post list, and a blog

Diff Detail

Repository
rP Phabricator
Branch
oblivious
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8650
Build 10020: Run Core Tests
Build 10019: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 34841.Nov 6 2015, 2:58 AM
chad retitled this revision from to Clean up Oblivious.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9546: Phame "Oblivious" skin a few rough edges.
Herald added a subscriber: Korvin. · View Herald TranscriptNov 6 2015, 2:58 AM
chad updated this revision to Diff 34842.Nov 6 2015, 3:00 AM
  • derp
chad added a comment.EditedNov 6 2015, 3:01 AM

I spent about 2 hours trying to "Markup" the Description and properly set the page's Title. I couldn't figure it out, very confusing how these get built.

chad retitled this revision from Clean up Oblivious to Clean up Oblivious skin.Nov 6 2015, 7:34 PM
epriestley accepted this revision.Nov 6 2015, 7:39 PM
epriestley edited edge metadata.

I can take a look at the description/title.

This revision is now accepted and ready to land.Nov 6 2015, 7:39 PM
Closed by commit rP3a90cb56b475: Clean up Oblivious skin (authored by chad, committed by chad). · Explain WhyNov 6 2015, 7:40 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Nov 6 2015, 7:44 PM

awwyiss

Revision Contents
Changeset List

PathSize
externals/
skins/
oblivious/
css/
36 lines
src/
applications/
phame/
skins/
23 lines
17 lines
storage/
2 lines
view/
4 lines

Diff 34841

View Options

externals/skins/oblivious/css/oblivious.css

Loading...
View Options

src/applications/phame/skins/PhameBasicBlogSkin.php

Loading...
View Options

src/applications/phame/skins/PhameBasicTemplateBlogSkin.php

Loading...
View Options

src/applications/phame/storage/PhameBlog.php

Loading...
View Options

src/applications/phame/view/PhamePostView.php

Loading...
Phabricator · Built by Phacility