Merge pull request #30 from srct/issue14

Fix Issue 14

schedules/app/assets/stylesheets/application.scss

@@ -383,7 +383,7 @@ footer {
         }
     }
 
-    /* This does nothing. 
+    /* This does nothing.
        I can't seem to find a way to edit these given the current HTML structure.
        If possible, I think that it would look more pleasing to make the background
        of the pop-up $white and the text contained inside of it $black, as it currently
@@ -397,4 +397,7 @@ footer {
         }
     }*/
 
+    :target{
+        background-color: $lightgreen;
+    }
 }

schedules/app/controllers/search_controller.rb

@@ -32,7 +32,9 @@ def index
     end
 
     /[0-9]{5}/.match(params[:query]) do |m|
-      redirect_to(course_url(CourseSection.latest_by_crn(m[0]).course))
+      course = CourseSection.latest_by_crn(m[0])
+      url = course_url(course.course) + "#section-" + course.id.to_s
+      redirect_to(url)
     end
 
     if @courses&.count == 1 && @instructors&.count&.zero?

schedules/app/views/courses/show.html.erb

@@ -1,5 +1,17 @@
 <%= render partial: 'shared/navbar' %>
 
+<script>
+    //turbolinks prevents the site from properly jumping to an element when using https://www.url.com/page#element-id
+    //additionally, CSS styling is not properly applied to any :target attributes
+    //this prevented issue 14 from being resolved (https://github.com/srct/schedules/issues/14)
+    //the following script forces the page to update both the CSS and the window's location *after* turbolinks does its thing.
+    document.addEventListener("turbolinks:load", function(){
+        if(window.location.hash){
+            window.location = window.location.hash;
+        }
+    });
+</script>
+
 <div class="row">
   <div class="col-12 col-lg">
     <h1><%= @course.full_name %></h1>