Allow "Magic Home" to be hidden
ClosedPublic
Actions

Authored by epriestley on Feb 1 2017, 6:01 PM.

Tags

None

Referenced Files

	F15448319: D17289.id41591.diff
	Fri, Mar 28, 4:11 AM

	F15437686: D17289.diff
	Tue, Mar 25, 9:01 PM

	F15425293: D17289.id41591.diff
	Sun, Mar 23, 3:51 AM

	F15389776: D17289.id41588.diff
	Sat, Mar 15, 5:38 AM

	F15380578: D17289.diff
	Fri, Mar 14, 3:42 AM

	Unknown Object (File)
	Sat, Mar 1, 3:27 PM

	Unknown Object (File)
	Feb 10 2025, 1:48 AM

	Unknown Object (File)
	Jan 31 2025, 10:37 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12174: Home Menu Errata

Commits

rP6abdae8e7396: Allow "Magic Home" to be hidden

Summary

Ref T12174. Fallback behavior on this already appears to be sensible.

Test Plan

Hid "Magic Home".
Viewed homepage with no dashboards on the menu.
Saw "Magic Home" content, with no item in the menu selected, which seems reasonable.

Screen Shot 2017-02-01 at 10.00.50 AM.png (780×1 px, 172 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 1 2017, 6:01 PM

Oh haha

🐈

Harbormaster completed remote builds in B15486: Diff 41588.Feb 1 2017, 6:02 PM

I can clean the UX up from here. I thought this would have been more difficult.

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

Yeah, I'm a little surprised the fallback already works since I also expected it to be a little more involved. Possible that it only appears to work and will need a little more effort, but should be easy to fix if that's true.

Closed by commit rP6abdae8e7396: Allow "Magic Home" to be hidden (authored by epriestley, committed by epriestley). · Explain WhyFeb 1 2017, 6:28 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

home/

menuitem/

PhabricatorHomeProfileMenuItem.php

5 lines

Diff 41591

View Options

src/applications/home/menuitem/PhabricatorHomeProfileMenuItem.php

Allow "Magic Home" to be hiddenClosedPublicActions