Page MenuHomePhabricator
Differential D16197

Misc Phame cleanup
ClosedPublic
Actions

Authored by chad on Jun 30 2016, 12:53 AM.
Tags
None
Referenced Files
F14828317: D16197.id.diff
Wed, Jan 29, 10:48 AM
Unknown Object (File)
Fri, Jan 24, 2:49 PM
Unknown Object (File)
Fri, Jan 24, 2:23 PM
Unknown Object (File)
Wed, Jan 22, 12:33 PM
Unknown Object (File)
Tue, Jan 21, 4:05 PM
Unknown Object (File)
Fri, Jan 17, 9:51 PM
Unknown Object (File)
Thu, Jan 16, 11:24 PM
Unknown Object (File)
Dec 15 2024, 5:29 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9360: Unbeta Phame
Commits
rPfd0a606f7942: Misc Phame cleanup
Summary

Ref T9360.

  • View Live useless on archived blogs
  • Edit Blog Image treatment like profiles
  • Pager next/prev should keep you on whatever view you're on
  • Unset user titles aren't falling back properly
  • Add captions to edit fields for better clarification
Test Plan

Archive a blog, Edit a photo, verify pager on live and internal blogs, check empty titles, and view new edit form instructions.

Diff Detail

Repository
rP Phabricator
Branch
phame-2 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12836
Build 16355: Run Core Tests
Build 16354: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 38959.Jun 30 2016, 12:53 AM
chad retitled this revision from to Misc Phame cleanup.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T9360: Unbeta Phame.
Herald added a subscriber: Korvin. · View Herald TranscriptJun 30 2016, 12:53 AM
epriestley accepted this revision.Jun 30 2016, 1:52 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 30 2016, 1:52 AM
Closed by commit rPfd0a606f7942: Misc Phame cleanup (authored by chad, committed by chad). · Explain WhyJun 30 2016, 2:08 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
controller/
blog/
13 lines
post/
10 lines
editor/
8 lines
storage/
12 lines

Diff 38959

View Options

src/applications/phame/controller/blog/PhameBlogManageController.php

Loading...
View Options

src/applications/phame/controller/post/PhamePostViewController.php

Loading...
View Options

src/applications/phame/editor/PhameBlogEditEngine.php

Loading...
View Options

src/applications/phame/storage/PhamePost.php

Loading...
Phabricator · Built by Phacility