From 7b12dac02116f36b8e7fd729c05ebdb326643188 Mon Sep 17 00:00:00 2001
From: marco <marcotuts@gmail.com>
Date: Tue, 22 Jan 2013 15:47:23 -0500
Subject: [PATCH] added make visible to menu, and visibility description to
 threads

---
 lms/static/sass/_discussion.scss                    | 8 +++++++-
 lms/templates/discussion/_new_post.html             | 8 ++++++++
 lms/templates/discussion/_thread_list_template.html | 1 -
 lms/templates/discussion/_underscore_templates.html | 1 +
 4 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/lms/static/sass/_discussion.scss b/lms/static/sass/_discussion.scss
index 9148421e4c..2d7e5a8354 100644
--- a/lms/static/sass/_discussion.scss
+++ b/lms/static/sass/_discussion.scss
@@ -169,6 +169,12 @@ body.discussion {
         }
       }
 
+      .form-group-label {
+        display: block;
+        padding-top: 5px;
+        color:#fff;
+      }
+
       .topic_dropdown_button {
         position: relative;
         z-index: 1000;
@@ -1373,7 +1379,7 @@ body.discussion {
 
   .group-visibility-label {
     font-size: 12px;
-    color:#ddd;
+    color:#fff;
     font-style: italic;
   }
 
diff --git a/lms/templates/discussion/_new_post.html b/lms/templates/discussion/_new_post.html
index 263c49eba1..7a0fa46e4a 100644
--- a/lms/templates/discussion/_new_post.html
+++ b/lms/templates/discussion/_new_post.html
@@ -45,6 +45,14 @@
                     %elif course.metadata.get("allow_anonymous_to_peers", False):
                         <input type="checkbox" name="anonymous_to_peers" class="discussion-anonymous-to-peers" id="new-post-anonymous-to-peers"><label for="new-post-anonymous-to-peers">post anonymously to classmates</label>
                     %endif
+                    <div class="form-group-label">
+                        Make visible to:
+                        <select class="group-filter-select"> 
+                            <option value="all">All Groups</option>
+                            <option value="g1">Group 1</option>
+                            <option value="g2">Group 2</option>
+                        </select>
+                    </div>
                 </div>
             </div>
             <div class="right-column">
diff --git a/lms/templates/discussion/_thread_list_template.html b/lms/templates/discussion/_thread_list_template.html
index 48aef4debf..31943ef8ad 100644
--- a/lms/templates/discussion/_thread_list_template.html
+++ b/lms/templates/discussion/_thread_list_template.html
@@ -24,7 +24,6 @@
             <option value="g1">Group 1</option>
             <option value="g2">Group 2</option>
         </select>
-
     </div>
     <div class="post-list-wrapper">
         <ul class="post-list">
diff --git a/lms/templates/discussion/_underscore_templates.html b/lms/templates/discussion/_underscore_templates.html
index be238811c2..105d677aba 100644
--- a/lms/templates/discussion/_underscore_templates.html
+++ b/lms/templates/discussion/_underscore_templates.html
@@ -26,6 +26,7 @@
   <div class="discussion-post">
       <div><a href="javascript:void(0)" class="dogear action-follow" data-tooltip="follow"></a></div>
       <header>
+          <div class="group-visibility-label">This post visible only to Group 1.</div>
           <a href="#" class="vote-btn discussion-vote discussion-vote-up" data-role="discussion-vote" data-tooltip="vote"><span class="plus-icon">+</span> <span class='votes-count-number'>${'<%- votes["up_count"] %>'}</span></a>
           <h1>${'<%- title %>'}</h1>
           <p class="posted-details">