• 
      

    Fix clicking on bare rows in datagrids.

    Review Request #10712 — Created Sept. 11, 2019 and submitted — Latest diff uploaded

    Information

    Djblets
    release-1.0.x
    028236e...

    Reviewers

    The datagrids once had a facility for the entire row to act as a link to
    the given object. Over time as we made changes, this functionality had
    been about halfway erased, making it so only cells with .has-link
    worked as links despite the entire row having a pointer icon. This
    change fixes up things so that the URL is set correctly on the tr
    element's data, and recreates a javascript handler for clicks that are
    otherwise unhandled.

    Clicked on various cells within different dashboards and checked to
    ensure that the correct page was loaded. For cells that contained their
    own link (for example, users), saw that the row-level link was ignored
    in favor of the cell-level link. Verified that clicks on
    interaction-type cells (stars, checkboxes) did not activate the link.