Switch spinners to use Font Awesome's fa-spinner.

Review Request #7173 — Created April 6, 2015 and submitted

Information

Review Board
release-2.5.x
83d4818...

Reviewers

Our old spinner.gif is one of the ugliest remaining parts of Review Board. It's
not retina-aware, and it feels pretty out of place in the modern world.

This change swaps out most of the uses of spinner.gif to use Font Awesome's
fa-spinner icon. This uses CSS transforms to rotate the image, and scales
cleanly to any size and resolution.

There are a couple more uses of the old spinner (in rich text rendering and the
new review request/upload diff UIs) which I have yet to clean up. I'll get to
those in a later change.

Looked at each piece of UI (often by making manual changes to the code to
ensure that the spinner stayed on screen long enough to really verify it).

reviewbot
  1. Tool: Pyflakes
    Ignored Files:
        reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js
        reviewboard/templates/base.html
        reviewboard/static/rb/css/pages/diffviewer.less
        reviewboard/static/rb/js/dashboard/views/dashboardView.js
        reviewboard/static/rb/js/utils/apiUtils.js
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/templates/diffviewer/view_diff.html
        reviewboard/static/rb/css/pages/base.less
        reviewboard/static/rb/css/pages/reviews.less
        reviewboard/static/rb/js/views/changeBoxView.js
        reviewboard/static/rb/css/common.less
        reviewboard/static/rb/js/common.js
        reviewboard/templates/admin/widgets/w-news.html
        reviewboard/static/rb/css/pages/dashboard.less
    
    
    
    Tool: PEP8 Style Checker
    Ignored Files:
        reviewboard/static/rb/js/diffviewer/views/diffFileIndexView.js
        reviewboard/templates/base.html
        reviewboard/static/rb/css/pages/diffviewer.less
        reviewboard/static/rb/js/dashboard/views/dashboardView.js
        reviewboard/static/rb/js/utils/apiUtils.js
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/templates/diffviewer/view_diff.html
        reviewboard/static/rb/css/pages/base.less
        reviewboard/static/rb/css/pages/reviews.less
        reviewboard/static/rb/js/views/changeBoxView.js
        reviewboard/static/rb/css/common.less
        reviewboard/static/rb/js/common.js
        reviewboard/templates/admin/widgets/w-news.html
        reviewboard/static/rb/css/pages/dashboard.less
    
    
  2. 
      
brennie
  1. 
      
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.5.x (9d166c5)
Loading...