HomePhabricator

Remove skins from Phame

Tags
None
Referenced Files
F1021634: Screen Shot 2015-12-11 at 7.11.12 AM.png
Dec 11 2015, 4:14 PM
F1021635: Screen Shot 2015-12-11 at 7.11.14 AM.png
Dec 11 2015, 4:14 PM
F1021638: Screen Shot 2015-12-11 at 7.11.21 AM.png
Dec 11 2015, 4:14 PM
Subscribers
None

Description

Remove skins from Phame

Summary:
Ref T9897. Purge a bunch of stuff:

  • Remove skins.
  • Remove all custom sites for skin resources.
  • Remove "framed", "notlive", "preview", separate "live" controllers (see below).
  • Merge "publish" and "unpublish" controllers into one.

New behavior:

  • Blogs and posts have three views:
    • "View": Internal view URI, which is a normal detail page.
    • "Internal Live": Internal view URI which is a little prettier.
    • "External Live": External view URI for an external domain.

Right now, the differences are pretty minor (basically, different crumbs/chrome). This mostly gives us room to put some milder flavor of skins back later (photography or more "presentation" elements, for example).

This removes 9 million lines of code so I probably missed a couple of things, but I think it's like 95% of the way there.

Test Plan:
Here are some examples of what the "view", "internal" and "external" views look like for blogs (posts are similar):

"View": Unchanged

Screen Shot 2015-12-11 at 7.11.12 AM.png (1×1 px, 124 KB)

"Internal": No chrome or footer. Still write actions (edit, post commments). Has crumbs to get back into Phame.

Screen Shot 2015-12-11 at 7.11.14 AM.png (1×1 px, 75 KB)

"External": No chrome or footer. No write actions. No Phabricator crumbs. No policy/status information.

Screen Shot 2015-12-11 at 7.11.21 AM.png (1×1 px, 68 KB)

I figure we'll probably tweak these a bit to figure out what makes sense (like: maybe no actions on "internal, live"? and "external, live" probably needs a way to set a root "Company >" crumb?) but that they're reasonable-ish as a first cut?

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9897

Differential Revision: https://secure.phabricator.com/D14740

Details

Provenance
epriestleyAuthored on
epriestleyPushed on Dec 11 2015, 4:14 PM
Reviewer
chad
Differential Revision
D14740: Remove skins from Phame
Parents
rPc62e0a10f6b3: Clarify that ApplicationEditor supports editing itself
Branches
Unknown
Tags
Unknown
Tasks
T9897: Unbeta Feedback on Phame
Build Status
Buildable 9470
Build 11285: Run Core Tests