Differential D17298

Remove menu item pinning from Home and Favorites profile menus
ClosedPublic
Actions

Authored by epriestley on Feb 2 2017, 3:40 PM.

Tags

None

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12174: Home Menu Errata

Commits

rPd182d7eaf1d1: Remove menu item pinning from Home and Favorites profile menus

Summary

Home now always uses the topmost item (falling back to "magic home") and no longer supports pinning. If any personal item may be a default item, it will always be picked over any global item.
Favorites doesn't use defaults anyway, but no longer has misleading UI suggesting it might.

Test Plan

Saw no pinning UI on Home/Favorites.
Added a personal dashboard on Home, it automatically became the new default.
Pinned stuff normally on Projects.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 2 2017, 3:40 PM

Harbormaster completed remote builds in B15503: Diff 41605.Feb 2 2017, 3:41 PM

epriestley mentioned this in T12174: Home Menu Errata.Feb 2 2017, 3:44 PM

chad accepted this revision.Feb 2 2017, 4:41 PM

This revision is now accepted and ready to land.Feb 2 2017, 4:41 PM

Closed by commit rPd182d7eaf1d1: Remove menu item pinning from Home and Favorites profile menus (authored by epriestley, committed by epriestley). · Explain WhyFeb 3 2017, 1:37 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

search/

engine/

PhabricatorProfileMenuEngine.php

43 lines

Diff 41617

src/applications/search/engine/PhabricatorProfileMenuEngine.php

Loading...