• 
      

    Redesign the sonar experience.

    Review Request #7534 — Created July 21, 2015 and submitted

    Information

    student-sonar
    master
    505224d...

    Reviewers

    This change is the start of a major redesign of how sonar is put together. I've
    brought in bootstrap, which gives us a lot of styles, layout, and various
    components for free.
    
    All of the students for the active terms are now listed in a sidebar, along
    with other "components" such as the calendar. This gives us a good place to put
    all the different features that I plan on building.

    Checked calendar and student views.


    Description From Last Updated

    showToGroups

    brenniebrennie

    Blank line between these.

    brenniebrennie

    Can you format this as ...(_.toArray(arguments) ? The splatting makes this look weird.

    brenniebrennie
    reviewbot
    1. Tool: PEP8 Style Checker
      Ignored Files:
          .jshintrc
          lib/frontend/collection-view.js
          lib/frontend/config.js
          lib/frontend/application.js
          lib/frontend/header-view.js
          lib/frontend/groups-view.js
          lib/api.js
          lib/frontend/main.js
          lib/frontend/index-view.js
          lib/frontend/sidebar-view.js
          lib/frontend/calendar-view.js
          lib/frontend/application-view.js
          lib/frontend/util.js
          package.json
          lib/frontend/status-report-checker-view.js
          lib/frontend/user-detail-view.js
          css/style.less
          lib/schema.js
          lib/frontend/models.js
          lib/frontend/group-by-collection.js
      
      
      
      Tool: Pyflakes
      Ignored Files:
          .jshintrc
          lib/frontend/collection-view.js
          lib/frontend/config.js
          lib/frontend/application.js
          lib/frontend/header-view.js
          lib/frontend/groups-view.js
          lib/api.js
          lib/frontend/main.js
          lib/frontend/index-view.js
          lib/frontend/sidebar-view.js
          lib/frontend/calendar-view.js
          lib/frontend/application-view.js
          lib/frontend/util.js
          package.json
          lib/frontend/status-report-checker-view.js
          lib/frontend/user-detail-view.js
          css/style.less
          lib/schema.js
          lib/frontend/models.js
          lib/frontend/group-by-collection.js
      
      
    2. 
        
    brennie
    1. 
        
    2. Show all issues

      showToGroups

    3. 
        
    david
    reviewbot
    1. Tool: Pyflakes
      Ignored Files:
          .jshintrc
          lib/frontend/collection-view.js
          lib/frontend/config.js
          lib/frontend/application.js
          lib/frontend/header-view.js
          lib/frontend/groups-view.js
          lib/api.js
          lib/frontend/main.js
          lib/frontend/index-view.js
          lib/frontend/sidebar-view.js
          lib/frontend/calendar-view.js
          lib/frontend/application-view.js
          lib/frontend/util.js
          package.json
          lib/frontend/status-report-checker-view.js
          lib/frontend/user-detail-view.js
          css/style.less
          lib/schema.js
          lib/frontend/models.js
          lib/frontend/group-by-collection.js
      
      
      
      Tool: PEP8 Style Checker
      Ignored Files:
          .jshintrc
          lib/frontend/collection-view.js
          lib/frontend/config.js
          lib/frontend/application.js
          lib/frontend/header-view.js
          lib/frontend/groups-view.js
          lib/api.js
          lib/frontend/main.js
          lib/frontend/index-view.js
          lib/frontend/sidebar-view.js
          lib/frontend/calendar-view.js
          lib/frontend/application-view.js
          lib/frontend/util.js
          package.json
          lib/frontend/status-report-checker-view.js
          lib/frontend/user-detail-view.js
          css/style.less
          lib/schema.js
          lib/frontend/models.js
          lib/frontend/group-by-collection.js
      
      
    2. 
        
    david
    brennie
    1. 
        
    2. lib/frontend/application-view.js (Diff revision 2)
       
       
       
      Show all issues

      Blank line between these.

    3. lib/frontend/application.js (Diff revision 2)
       
       
      Show all issues

      Can you format this as ...(_.toArray(arguments) ? The splatting makes this look weird.

    4. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to master (6bea534)