Homestyx hydra
Diffusion hydra 36a8b4ea1796

When a ProfileMenu has a link item that adds URI parameters, highlight it when…

Description

When a ProfileMenu has a link item that adds URI parameters, highlight it when clicked

Summary:
Depends on D20352. Fixes T12949. If a user adds a link (for example, to a workboard) that takes you to the same page but with some URI parameters, we'd prefer to highlight the "link" item instead of the default "workboard" item when you click it.

For example, you add a /query/assigned/ "link" item to a workboard, called "Click This To Show Tasks Assigned To Me On This Workboard", i.e. filter the current view.

This is a pretty reasonable thing to want to do. When you click it, we'd like to highlight that item to show that you've activated the "Assigned to Me" filter you added.

However, we currently highlight the thing actually serving the content, i.e. the "Workboard" item.

Instead:

  • When picking what to highlight, look through all the items for one with a link to the current request URI.
  • If we find one or more, pick the one that would be the default.
  • Otherwise, pick the first one.

This means that you can have several items like "?a=1", "?a=2", etc., and we will highlight them correctly when you click them.

This actual patch has some questionable bits (see some discussion in T13275), but I'd like to wait for stronger motivation to refactor it more extensively.

Test Plan:

  • On a portal, added a ?a=1 link. Saw it highlight properly when clikced.
  • On a workboard, added a link to the board itself with a different filter. Saw it highlight appropriately when clicked.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T12949

Differential Revision: https://secure.phabricator.com/D20353

Details

Provenance
epriestleyAuthored on Mar 30 2019, 2:52 PM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:408cbd633cd5: On portals, make the "selected" / "default" logic more straightforward
Branches
Unknown
Tags
Unknown

Event Timeline