Page MenuHomePhabricator

Build Badges View page into more of a profile
ClosedPublic

Authored by chad on Feb 15 2017, 5:54 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 10, 3:45 AM
Unknown Object (File)
Feb 24 2024, 11:44 PM
Unknown Object (File)
Feb 12 2024, 10:05 PM
Unknown Object (File)
Feb 3 2024, 2:36 PM
Unknown Object (File)
Jan 27 2024, 9:51 AM
Unknown Object (File)
Jan 27 2024, 12:03 AM
Unknown Object (File)
Jan 20 2024, 8:39 PM
Unknown Object (File)
Jan 13 2024, 3:26 PM
Subscribers

Details

Summary

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

Test Plan

Review badge and recipients in sandbox.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad edited the summary of this revision. (Show Details)
  • fix create transaction
  • "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.

This revision was automatically updated to reflect the committed changes.