Remove sidebar class from forum nav

This will enable much cleanup of the relevant SASS rules.

common/static/coffee/spec/discussion/view/discussion_thread_list_view_spec.coffee

@@ -147,7 +147,7 @@ describe "DiscussionThreadListView", ->
                 </div>
             </div>
         </script>
-        <div class="sidebar"></div>
+        <div class="forum-nav"></div>
         """
         @threads = [
           makeThreadWithProps({
@@ -176,7 +176,7 @@ describe "DiscussionThreadListView", ->
         spyOn($, "ajax")
 
         @discussion = new Discussion([])
-        @view = new DiscussionThreadListView({collection: @discussion, el: $(".sidebar")})
+        @view = new DiscussionThreadListView({collection: @discussion, el: $(".forum-nav")})
         @view.render()
 
     makeThreadWithProps = (props) ->
@@ -196,7 +196,7 @@ describe "DiscussionThreadListView", ->
 
     makeView = (discussion) ->
       return new DiscussionThreadListView(
-          el: $(".sidebar"),
+          el: $(".forum-nav"),
           collection: discussion
       )
 

common/static/coffee/src/discussion/discussion_router.coffee

@@ -8,7 +8,7 @@ if Backbone?
         @discussion = options['discussion']
         @course_settings = options['course_settings']
 
-        @nav = new DiscussionThreadListView(collection: @discussion, el: $(".sidebar"))
+        @nav = new DiscussionThreadListView(collection: @discussion, el: $(".forum-nav"))
         @nav.on "thread:selected", @navigateToThread
         @nav.on "thread:removed", @navigateToAllThreads
         @nav.on "threads:rendered", @setActiveThread

common/static/coffee/src/discussion/views/discussion_thread_list_view.coffee

@@ -89,7 +89,7 @@ if Backbone?
       discussionsBodyTop = if discussionBody[0] then discussionBody.offset().top
       discussionsBodyBottom = discussionsBodyTop + discussionBody.outerHeight()
 
-      sidebar = $(".sidebar")
+      sidebar = $(".forum-nav")
       if scrollTop > discussionsBodyTop - @sidebar_padding
         sidebar.css('top', scrollTop - discussionsBodyTop + @sidebar_padding);
       else

common/test/acceptance/pages/lms/discussion.py

@@ -343,7 +343,7 @@ class DiscussionUserProfilePage(CoursePage):
 
 class DiscussionTabHomePage(CoursePage, DiscussionPageMixin):
 
-    ALERT_SELECTOR = ".discussion-body .sidebar .search-alert"
+    ALERT_SELECTOR = ".discussion-body .forum-nav .search-alert"
 
     def __init__(self, browser, course_id):
         super(DiscussionTabHomePage, self).__init__(browser, course_id)

lms/static/sass/discussion/_discussion.scss

@@ -436,7 +436,6 @@ body.discussion {
   }
 
   section.user-profile {
-    @extend .sidebar;
     display: table-cell;
     border-right: 1px solid #ddd;
     border-radius: 3px 0 0 3px;

lms/static/sass/discussion/elements/_navigation.scss

 .forum-nav {
   @include box-sizing(border-box);
   float: left;
+  position: relative;
   border: 1px solid #aaa;
   border-radius: 3px;
 }
@@ -27,6 +28,7 @@
   }
 
   .icon {
+    @include font-size(14);
     margin-right: ($baseline/4);
   }
 }
@@ -87,7 +89,17 @@
 }
 
 .forum-nav-browse-menu {
+  @include font-size(14);
   overflow-y: scroll;
+  list-style: none;
+}
+
+.forum-nav-browse-submenu {
+  list-style: none;
+}
+
+.forum-nav-browse-title {
+  display: block;
 }
 
 .forum-nav-browse-title .icon {
@@ -130,6 +142,7 @@
 // -----------
 .forum-nav-thread-list {
   overflow-y: scroll;
+  list-style: none;
 }
 
 .forum-nav-thread {
@@ -137,7 +150,7 @@
 }
 
 .forum-nav-thread-link {
-  @include clearfix();
+  display: block;
 }
 
 %forum-nav-thread-wrapper {
@@ -269,6 +282,7 @@
 }
 
 %forum-nav-load-more-content {
+  display: block;
   text-align: center;
 }
 

lms/static/sass/discussion/utilities/_developer.scss

@@ -19,7 +19,7 @@
 // --------------------
 body.discussion {
 
-  .sidebar {
+  .forum-nav {
 
     // wrapper for multiple alerts
     .search-alerts {

lms/static/sass/discussion/utilities/_shame.scss

@@ -62,6 +62,12 @@
   font-size: 12px !important;
 }
 
+// Override global ul rules
+.forum-nav-browse-menu, .forum-nav-browse-submenu {
+  margin: 0;
+  padding-left: 0;
+}
+
 // The sidebar class does a lot of things that we don't want in the thread list;
 // the following rules contain styling that is necessary and would otherwise
 // reside in elements/_navigation.scss if the sidebar styling did not make the
@@ -138,6 +144,12 @@ li[class*=forum-nav-thread-label-] {
 // The following rules would be unnecessary but for broadly scoped rules defined
 // elsewhere in our CSS.
 
+// Override global ul rules
+.forum-nav-thread-list, .forum-nav-thread-labels {
+  margin: 0;
+  padding-left: 0;
+}
+
 li[class*=forum-nav-thread-label-] {
   // Override global span rules
   span {

lms/templates/discussion/index.html

@@ -37,7 +37,7 @@
          data-user-cohort-id="${user_cohort}"
          data-course-settings="${course_settings}">
     <div class="discussion-body">
-        <div class="sidebar forum-nav"></div>
+        <div class="forum-nav"></div>
         <div class="discussion-column">
         </div>
     </div>