Make some of the datagrid classes more generically useful.

Review Request #7260 — Created April 28, 2015 and submitted

Information

Review Board
release-2.5.x
2df2f83...

Reviewers

The recent dashboard class revamp renamed much of the dashboard-specific
classes to datagrid-specific classes, so we could use them on more
pages. This change goes a step further, taking many of those classes and
making them useful outside of datagrids.

There are now a couple of top-level classes for <body> that define
whether the page has a sidebar or whether the content will be taking up
the full visible height of the page.

Some of the datagrid page IDs (#datagrid_page_container and
#datagrid_page_content) have been removed, instead making of use of
#page-container, #content_container, and #content. That's two layers of
elements and styles that are removed and made generic.

The sidebar classes are also now generic, no longer tied to datagrids.
This will allow us to reuse it for the New Review Request page, and
others.

Tested all the datagrid pages in desktop and mobile modes. Made sure
fit-to-viewport, scrolling, and resizing all worked as expected.

Tested the review request, diff viewer, and admin pages.

reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/base.html
        reviewboard/templates/datagrids/sidebar_section.html
        reviewboard/static/rb/css/ui/sidebars.less
        reviewboard/static/rb/css/pages/base.less
        reviewboard/static/rb/css/ui/datagrids.less
        reviewboard/templates/datagrids/sidebar_nav_item.html
        reviewboard/templates/datagrids/sidebar_user_info.html
        reviewboard/static/rb/css/defs.less
        reviewboard/static/rb/js/pages/views/datagridPageView.js
    
    
    
    Tool: Pyflakes
    Processed Files:
        reviewboard/staticbundles.py
    
    Ignored Files:
        reviewboard/templates/datagrids/datagrid.html
        reviewboard/templates/base.html
        reviewboard/templates/datagrids/sidebar_section.html
        reviewboard/static/rb/css/ui/sidebars.less
        reviewboard/static/rb/css/pages/base.less
        reviewboard/static/rb/css/ui/datagrids.less
        reviewboard/templates/datagrids/sidebar_nav_item.html
        reviewboard/templates/datagrids/sidebar_user_info.html
        reviewboard/static/rb/css/defs.less
        reviewboard/static/rb/js/pages/views/datagridPageView.js
    
    
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.5.x (10a1734)
Loading...