search

adicu / Courses

A course management application for Columbia.
courses.adicu.com
Other
16 stars 12 forks source link

Highlight selected courses #103

Closed rbtying closed 10 years ago

rbtying commented 10 years ago

Fixes the handling of the section isSelected() flag.

brishin commented 10 years ago

Getting on the subway but will take a look tonight.

On Monday, November 11, 2013, Robert Ying wrote:

Fixes the handling of the section isSelected() flag.

You can merge this Pull Request by running

git pull https://github.com/adicu/Courses highlight_selected_courses

Or view, comment on, or merge it at:

https://github.com/adicu/Courses/pull/103 Commit Summary

  • Set section selected flag properly
  • Support multi-letter prefixes

File Changes

  • M public/partials/directives/course-popover.htmlhttps://github.com/adicu/Courses/pull/103/files#diff-0(2)
  • M src/css/app.lesshttps://github.com/adicu/Courses/pull/103/files#diff-1(3)
  • M src/js/directives/courseBlock.coffeehttps://github.com/adicu/Courses/pull/103/files#diff-2(2)
  • M src/js/filters.coffeehttps://github.com/adicu/Courses/pull/103/files#diff-3(2)
  • M src/js/models/Course.coffeehttps://github.com/adicu/Courses/pull/103/files#diff-4(4)
  • M src/js/models/Schedule.coffeehttps://github.com/adicu/Courses/pull/103/files#diff-5(8)

Patch Links: