If a ProfileMenu has Link items, and one or more have the request URI as the link target, highlight the first one
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	chad
	Aug 2 2017, 4:22 PM

Description

If users add "Link" menu items to a profile menu that point to variations of the object itself, they don't highlight when followed, but it would be more clear to highlight them.

For example, on Workboards, users may add Link items which point to different views of the workboard (open tasks, closed tasks, etc). When clicked, these will show the right content but the "Workboard" item will remain highlighted. When the request URI exactly matches one or more targets of Link items, highlighting the first Link item is a better behavior than highlighting the Workboard item.

Original Description

User is requesting saving workboard queries and exposing them via menu item, to click and see different views quickly. I think this sounds great, but it breaks my little design head when I think about implementation.

Revisions and Commits

rP Phabricator
	Abandoned		D18821 WIP Menu item that is a link to a state of a builtin menu item should display consistent selection behavior
	Abandoned		D18346 Override project nav menu filter if link item matching URI exists
		D20353	rP36a8b4ea1796 When a ProfileMenu has a link item that adds URI parameters, highlight it when…

Related Objects

Mentioned In: rP950e9d085b48: In ProfileMenu, put more structure between "stored/configured items" and…
rP408cbd633cd5: On portals, make the "selected" / "default" logic more straightforward
D20357: In ProfileMenu, put more structure between "stored/configured items" and "display items"
D20352: On portals, make the "selected" / "default" logic more straightforward
T13275: Build Portals: a standalone menu page / dashboard of dashboards
T13272: Remove the hard-coded homepage, and other Dashboards improvements
T13083: Plans: Portals and Facts
Mentioned Here: T12374: Saved Workboard Queries

Event Timeline

chad triaged this task as Wishlist priority.Aug 2 2017, 4:22 PM

chad created this task.

chad added projects: Feature Request, Workboards.

Herald added a subscriber: eadler. · View Herald TranscriptAug 2 2017, 4:22 PM

What's the root problem?
Why can't they use links?
How is this different from T12374?

A link doesn't stay highlighted if clicked on. Like dashboards, I can understand wanting multiple board views from the sidebar.

Or maybe there is a simple way to get the link to highlight, I couldn't think of one though.

Is there any reason that the rule "if any menu items have the same link target as the request URI, highlight the first one" wouldn't work?

That sounds much easier than all the solutions I came up with.

epriestley renamed this task from Build a Workboard Query MenuItem? to If a ProfileMenu has Link items, and one or more have request URI as the link target, highlight the first one.Aug 2 2017, 4:40 PM

Heh, that was going to be my suggestion in the original discourse post, but I figured it was too simplistic. Glad to hear we're on the same page. Thanks everyone!

epriestley renamed this task from If a ProfileMenu has Link items, and one or more have request URI as the link target, highlight the first one to If a ProfileMenu has Link items, and one or more have the request URI as the link target, highlight the first one.Aug 2 2017, 4:43 PM

epriestley updated the task description. (Show Details)

I'm obviously low on AM brian power today.

This is an unusually low-tech solution but I can't think of any reasons why it won't work.

chad edited projects, added Profile Menu Item; removed Workboards.Aug 2 2017, 5:19 PM

I've written a patch to make this work for the project sidebar nav for my organization's install. I don't want to take up your time with it if it's not helpful, but let me know if you'd like me to push a diff.

Happy to take a look if you found a reasonable way to get access to $request. If you're reading $_REQUEST['__path__'] directly or something though the eventual upstream version probably won't look too similar.