Differential D17358

Build Badges View page into more of a profile
ClosedPublic
Actions

Authored by chad on Feb 15 2017, 5:54 PM.

Tags

None

Referenced Files

	F14105539: D17358.diff
	Wed, Nov 27, 2:54 AM

	F14100594: D17358.id41742.diff
	Tue, Nov 26, 2:25 PM

	Unknown Object (File)
	Sat, Nov 23, 3:44 PM

	Unknown Object (File)
	Sat, Nov 23, 2:07 PM

	Unknown Object (File)
	Tue, Nov 19, 12:57 PM

	Unknown Object (File)
	Fri, Nov 15, 5:08 AM

	Unknown Object (File)
	Thu, Nov 14, 3:47 AM

	Unknown Object (File)
	Wed, Nov 13, 2:55 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T10798: Limit, order recipients list in BadgeView

Commits

rP9716e83d605f: Build Badges View page into more of a profile

Summary

Ref T10798. Cleans up the UI a little and adds a sidenav.

Test Plan

Review badge and recipients in sandbox.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Feb 15 2017, 5:54 PM

Herald added a subscriber: Korvin. · View Herald TranscriptFeb 15 2017, 5:54 PM

Harbormaster completed remote builds in B15638: Diff 41742.Feb 15 2017, 5:56 PM

chad edited the summary of this revision. (Show Details)Feb 15 2017, 6:03 PM

chad added a task: T12269: New badge transaction says "created this object"..

fix create transaction

Harbormaster completed remote builds in B15639: Diff 41743.Feb 15 2017, 6:03 PM

Harbormaster completed remote builds in B15639: Diff 41743.

remove needRecipients

Harbormaster completed remote builds in B15640: Diff 41744.Feb 15 2017, 6:09 PM

"BADGE" label in the menu maybe slightly weird/inconsistent? e..g, projects don't say "PROJECT".
"Add Recipients" uses language more like "Award Badge" elsewhere, maybe worth adjusting?
After adding recipients, take viewer to recipients page?
Put "Add recipients" action on recipients page?

This revision is now accepted and ready to land.Feb 15 2017, 6:55 PM

"BADGE" label in the menu maybe slightly weird/inconsistent? e..g, projects don't say "PROJECT".

we do this for Dashboards, and on Profiles we have a photo. Ideally, I'd always have some identifier before the menu, but it might take a few passes to find something consistent.

Closed by commit rP9716e83d605f: Build Badges View page into more of a profile (authored by chad, committed by chad). · Explain WhyFeb 15 2017, 7:32 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

7 lines

applications/

badges/

application/

PhabricatorBadgesApplication.php

13 lines

controller/

PhabricatorBadgesEditRecipientsController.php

6 lines

PhabricatorBadgesProfileController.php

88 lines

PhabricatorBadgesRecipientsController.php

58 lines

PhabricatorBadgesViewController.php

42 lines

storage/

PhabricatorBadgesBadge.php

9 lines

PhabricatorBadgesTransaction.php

4 lines

view/

PhabricatorBadgesRecipientsListView.php

14 lines

Diff 41749

src/__phutil_library_map__.php

Loading...

src/applications/badges/application/PhabricatorBadgesApplication.php

Loading...

src/applications/badges/controller/PhabricatorBadgesEditRecipientsController.php

Loading...

src/applications/badges/controller/PhabricatorBadgesProfileController.php

Loading...

src/applications/badges/controller/PhabricatorBadgesRecipientsController.php

Loading...

src/applications/badges/controller/PhabricatorBadgesViewController.php

Loading...

src/applications/badges/storage/PhabricatorBadgesBadge.php

Loading...

src/applications/badges/storage/PhabricatorBadgesTransaction.php

Loading...

src/applications/badges/view/PhabricatorBadgesRecipientsListView.php

Loading...