• 
      

    Fix the Edit Columns button after updating a datagrid.

    Review Request #8382 — Created Sept. 5, 2016 and submitted — Latest diff uploaded

    Information

    Djblets
    release-0.9.x
    795032e...

    Reviewers

    The Edit Columns button on datagrids was breaking after reordering
    columns, and likely after other operations that would result in datagrid
    state being reloaded. This was due to the Edit Columns element
    disappearing, and the event handler being invalidated as a result (as it
    was only set up after handling mobile mode changes, or when initially
    creating the datagrid).

    We now attach the event handler higher up, allowing the event to bubble,
    so we don't have to micromanage it. We also reload the ID both when
    rebuilding the header for a mobile mode change and when building the
    header after reload of datagrid contents.

    Checked the button after loading the datagrid, and saw the menu appear.

    Reordered columns (as per the repro case) and checked the button again.
    It worked.

    Went into mobile mode (by resizing the window), and checked. Still worked.

    Went out of mobile mode, and checked once more. Worked again.

    Repeated these tests but with adding columns.