Skip to content
  • epriestley's avatar
    On Dashboard tab panels in edit mode, make the "Tab Name" and the "Dropdown... · 9ad9ac9b
    epriestley authored
    On Dashboard tab panels in edit mode, make the "Tab Name" and the "Dropdown Edit Caret" into different links
    
    Summary:
    Ref T13272. In edit mode, tab panels now have a dropdown menu. However, this sort of overrlaps with the actual action of clicking the tab to select it.
    
    Separate these into different click targets so that "select tab X" and "open dropdown menu for X" are different operations.
    
    This is more work than it appears because:
    
      - We have an "action icon" already, used when you put a dashboard on a portal/home to create an "Edit" link. It makes sense to attach dropdowns to this, but it has some hard-coded stuff.
      - In applications with a "Create <thing>" in the crumbs (like Maniphest), we may use a dropdown menu if there are multiple create forms available. However, this menu renders in a weird way by reading all the properties out of an actual "View" object and building something else.
      - The "list of tabs" stuff shares code with different "list of tabs" navigation used by Diffusion and Instances.
    
    ..but I think I fixed everything and didn't break anything.
    
    Test Plan:
      - Clicked "select tab" and "open dropdown menu" as separate actions.
      - Viewed Diffusion, Maniphest with multiple create forms, Instances.
    
    Reviewers: amckinley
    
    Reviewed By: amckinley
    
    Maniphest Tasks: T13272
    
    Differential Revision: https://secure.phabricator.com/D20396
    9ad9ac9b