• 
      

    Give all datagrid pages access to some dashboard-level functionality.

    Review Request #7225 — Created April 17, 2015 and submitted

    Information

    Review Board
    release-2.5.x
    13b9151...

    Reviewers

    The dashboard page was the only one that could auto-size to fit the
    page, auto-reload content from the server, or set up any multi-select
    actions. This was all due to the DashboardView class powering it in
    JavaScript.

    That class has since been pulled out into a more reusable
    DatagridPageView and accompanying DatagridPageModel. These control the
    auto-sizing, and allow datagrids to take advantage of actions and
    periodic reloads, along with other events.

    The dashboard is still the only one with multi-select actions, but all
    the other pages now auto-fit, which looks far nicer than what we had
    before. In time, we may decide to port other features to these
    datagrids, now that we can do so more easily.

    Since all datagrids now fit the dimensions of the page, the special
    class for a scrollable sidebar has been removed. That mode is now the
    default.

    Tested each datagrid page we have. Saw that they were all nicely fitting
    to the page dimensions.

    Tested that the sidebars were doing the right thing on the dashboard and
    user pages.

    Tested dashboard actions.


    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/datagrids/grids.py
          reviewboard/staticbundles.py
      
      Ignored Files:
          reviewboard/static/rb/js/pages/models/dashboardModel.js
          reviewboard/static/rb/js/pages/views/dashboardView.js
          reviewboard/templates/datagrids/datagrid.html
          reviewboard/static/rb/js/dashboard/views/dashboardView.js
          reviewboard/templates/datagrids/dashboard.html
          reviewboard/static/rb/css/ui/datagrids.less
          reviewboard/static/rb/css/defs.less
          reviewboard/static/rb/js/pages/views/datagridPageView.js
          reviewboard/static/rb/js/pages/models/datagridPageModel.js
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/datagrids/grids.py
          reviewboard/staticbundles.py
      
      Ignored Files:
          reviewboard/static/rb/js/pages/models/dashboardModel.js
          reviewboard/static/rb/js/pages/views/dashboardView.js
          reviewboard/templates/datagrids/datagrid.html
          reviewboard/static/rb/js/dashboard/views/dashboardView.js
          reviewboard/templates/datagrids/dashboard.html
          reviewboard/static/rb/css/ui/datagrids.less
          reviewboard/static/rb/css/defs.less
          reviewboard/static/rb/js/pages/views/datagridPageView.js
          reviewboard/static/rb/js/pages/models/datagridPageModel.js
      
      
    2. 
        
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.5.x (abe5a97)