Differential D20362

Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single mode
ClosedPublic
Actions

Authored by epriestley on Apr 1 2019, 4:25 PM.

Details

Reviewers

amckinley

Maniphest Tasks

T13272: Remove the hard-coded homepage, and other Dashboards improvements

Commits

rPeea093bec816: Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single…

Summary

Depends on D20361. Ref T13272. Currently, Dashboards have three separate modes: view, arrange, manage.

With the advent of Portals, I think we can simplify this, and make the dashboard view a combined view/edit/manage page. To view it in a cleaner standalone way, you can add it to a portal/home/project. I'll also improve the "Install" workflow.

Test Plan

Viewed a dashboard page, clicked through all the actions, grepped for affected URIs.

Diff Detail

Repository

rP Phabricator

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Apr 1 2019, 4:25 PM

Harbormaster completed remote builds in B22455: Diff 48582.Apr 1 2019, 4:27 PM

epriestley requested review of this revision.Apr 1 2019, 4:27 PM

epriestley added a child revision: D20364: Make the "Install Dashboard" flow smoother.Apr 1 2019, 8:10 PM

amckinley accepted this revision.Apr 2 2019, 9:05 PM

This revision is now accepted and ready to land.Apr 2 2019, 9:05 PM

Closed by commit rPeea093bec816: Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single… (authored by epriestley). · Explain WhyApr 9 2019, 8:32 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

		Path
M		src/__phutil_library_map__.php (12 lines)
M		src/applications/dashboard/application/PhabricatorDashboardApplication.php (2 lines)
V		src/applications/dashboard/controller/{ → dashboard}/PhabricatorDashboardArchiveController.php
D	M	src/applications/dashboard/controller/PhabricatorDashboardArrangeController.php (79 lines)
V		src/applications/dashboard/controller/{ → dashboard}/PhabricatorDashboardEditController.php
V		src/applications/dashboard/controller/{ → dashboard}/PhabricatorDashboardInstallController.php
V		src/applications/dashboard/controller/{PhabricatorDashboardManageController.php → dashboard/PhabricatorDashboardViewController.php}
M		src/applications/dashboard/controller/PhabricatorDashboardProfileController.php (46 lines)
M		src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php (2 lines)
D	M	src/applications/dashboard/controller/PhabricatorDashboardViewController.php (68 lines)
V		src/applications/dashboard/controller/{dashboard ← }/PhabricatorDashboardArchiveController.php (2 lines)
V		src/applications/dashboard/controller/{dashboard ← }/PhabricatorDashboardEditController.php (11 lines)
V		src/applications/dashboard/controller/{dashboard ← }/PhabricatorDashboardInstallController.php (3 lines)
V		src/applications/dashboard/controller/{dashboard/PhabricatorDashboardViewController.php ← PhabricatorDashboardManageController.php} (70 lines)
M		src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php (9 lines)
M		src/applications/dashboard/storage/PhabricatorDashboard.php (7 lines)
M		src/applications/search/menuitem/PhabricatorDashboardProfileMenuItem.php (2 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	48582	c7f96d5		Apr 1 2019, 4:25 PM	★	★
Diff 2	48631	7e1743a	rPeea093bec816ab748ed7caddb606255d37a51b03	Apr 9 2019, 8:32 PM	★	★

Status	Author	Revision
Closed	epriestley	D20385 Fix some overlooked profile menu construction callsites
Closed	epriestley	D20384 Rebuild Dashboards on EditEngine: v1 Major Jank Edition
Closed	epriestley	D20383 When a dropdown menu would render in a way that hides it offscreen, try a different alignment
Closed	epriestley	D20382 Rebuild query panels on top of EditEngine
Closed	epriestley	D20377 When a dashboard has inconsistent panel policies: keep doing nothing
Closed	epriestley	D20376 On Dashboards, distinguish between invalid panels and restricted panels
Closed	epriestley	D20374 Make "Favorites" work more like other customizable menus
Closed	epriestley	D20373 Make the Dashboard dropshadow a little lighter and turn panel management into a menu
Closed	epriestley	D20372 Use EditEngine, not CustomFields, to define Dashboard Panel edit behavior
Closed	epriestley	D20371 Mostly replace the old panel "Edit" controller with the new "Editpro" controller
Closed	epriestley	D20370 Modularize Dashboard Panel transactionns
Closed	epriestley	D20369 Move Dashboard panel controllers into a "panel/" subdirectory
Closed	epriestley	D20368 Put "workflow" on Dashboard edit links when they're disabled
Closed	epriestley	D20367 Remove the dashboard "template" workflow
Closed	epriestley	D20364 Make the "Install Dashboard" flow smoother
Closed	epriestley	D20362 Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single mode
Closed	epriestley	D20361 Make Portals reachable without knowing the URI
Closed	epriestley	D20360 Fix the incorrect link target for "Create Revision" as a Menu Item
Closed	epriestley	D20359 When picking a default menu item to render, don't pick disabled items
Closed	epriestley	D20358 Update all existing ProfileMenuItems for the more-structured API
Closed	epriestley	D20357 In ProfileMenu, put more structure between "stored/configured items" and "display items"
Closed	epriestley	D20356 Automatically build mobile menus from navigation, and clean up external ProfileMenu API
Closed	epriestley	D20355 Allow profile menu items to be locked to the top or bottom of the menu
Closed	epriestley	D20353 When a ProfileMenu has a link item that adds URI parameters, highlight it when clicked
Closed	epriestley	D20352 On portals, make the "selected" / "default" logic more straightforward
Closed	epriestley	D20349 Allow Portals to be edited, and improve empty/blank states
Closed	epriestley	D20348 Add skeleton for Portals, a collection of dashboards and other resources

Diff 48631

View Options

src/__phutil_library_map__.php

View Options

src/applications/dashboard/application/PhabricatorDashboardApplication.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardArchiveController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardArrangeController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardEditController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardInstallController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardManageController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardProfileController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php

View Options

src/applications/dashboard/controller/PhabricatorDashboardViewController.php

View Options

src/applications/dashboard/controller/dashboard/PhabricatorDashboardArchiveController.php

View Options

src/applications/dashboard/controller/dashboard/PhabricatorDashboardEditController.php

View Options

src/applications/dashboard/controller/dashboard/PhabricatorDashboardInstallController.php

View Options

src/applications/dashboard/controller/dashboard/PhabricatorDashboardViewController.php

View Options

src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php

View Options

src/applications/dashboard/storage/PhabricatorDashboard.php

View Options

src/applications/search/menuitem/PhabricatorDashboardProfileMenuItem.php

Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single modeClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 48631

src/__phutil_library_map__.php

src/applications/dashboard/application/PhabricatorDashboardApplication.php

src/applications/dashboard/controller/PhabricatorDashboardArchiveController.php

src/applications/dashboard/controller/PhabricatorDashboardArrangeController.php

src/applications/dashboard/controller/PhabricatorDashboardEditController.php

src/applications/dashboard/controller/PhabricatorDashboardInstallController.php

src/applications/dashboard/controller/PhabricatorDashboardManageController.php

src/applications/dashboard/controller/PhabricatorDashboardProfileController.php

src/applications/dashboard/controller/PhabricatorDashboardQueryPanelInstallController.php

src/applications/dashboard/controller/PhabricatorDashboardViewController.php

src/applications/dashboard/controller/dashboard/PhabricatorDashboardArchiveController.php

src/applications/dashboard/controller/dashboard/PhabricatorDashboardEditController.php

src/applications/dashboard/controller/dashboard/PhabricatorDashboardInstallController.php

src/applications/dashboard/controller/dashboard/PhabricatorDashboardViewController.php

src/applications/dashboard/query/PhabricatorDashboardSearchEngine.php

src/applications/dashboard/storage/PhabricatorDashboard.php

src/applications/search/menuitem/PhabricatorDashboardProfileMenuItem.php

Combine the "View", "Arrange", and "Manage" modes of Dashboards into a single mode
ClosedPublic
Actions

Revision Contents
Changeset List