• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-2.5.x (9d166c5)