Differential D15449

Convert people manage page to two column
ClosedPublic
Actions

Authored by chad on Mar 9 2016, 4:52 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T10545: People and Projects are no longer flaggable

Commits

Restricted Diffusion Commit
rP1392872c5c03: Convert people manage page to two column

Summary

Ref T10545, this brings flags back? and converts the layout to two column w/curtain

Test Plan

View a few manage pages.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37231.Mar 9 2016, 4:52 PM

chad retitled this revision from to Convert people manage page to two column.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

chad added a task: T10545: People and Projects are no longer flaggable.

Herald added a subscriber: Korvin. · View Herald TranscriptMar 9 2016, 4:52 PM

chad added inline comments.Mar 9 2016, 4:53 PM

src/applications/people/controller/PhabricatorPeopleProfileManageController.php
97–98	is this still actually needed? I've seen it left in other conversions.

epriestley accepted this revision.Mar 9 2016, 4:55 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/people/controller/PhabricatorPeopleProfileManageController.php
97–98	No. If I left it, it was accidental.

This revision is now accepted and ready to land.Mar 9 2016, 4:55 PM

Closed by commit rP1392872c5c03: Convert people manage page to two column (authored by chad, committed by chad). · Explain WhyMar 9 2016, 4:57 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

controller/

PhabricatorPeopleProfileManageController.php

48 lines

PhabricatorPeopleProfileViewController.php

1 line

Diff 37232

View Options

src/applications/people/controller/PhabricatorPeopleProfileManageController.php