Redesign the sonar experience.

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

david
student-sonar
master
7527
7536
505224d...
student-sonar
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. 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)
     
     
     

    Blank line between these.

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

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

  4. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (6bea534)
Loading...