Merge pull request #3130 from hypothesis/sheetaluk/303-add-explanatio…

…n-text-to-the-sharer-on-only-me-and-group-annotations Add message to private and group annotations sharer.
hypothesis · Mar 24, 2016 · 28bb1ee · 28bb1ee
2 parents d8a8627 + 747f158
commit 28bb1ee
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/h/static/styles/annotation.scss b/h/static/styles/annotation.scss
@@ -180,7 +180,6 @@ $annotation-card-left-padding: 10px;
   background: $white;
   border: 1px solid $gray-lighter;
   border-radius: 2px;
-  white-space: nowrap;
   width: 200px;
   font-size: $body1-font-size;
   cursor: default;
@@ -224,6 +223,17 @@ $annotation-card-left-padding: 10px;
   color: $text-color;
 }
 
+.share-dialog-msg {
+  color: $gray-light;
+  margin: -5px 10px 10px 10px;
+  line-height: 15px;
+  font-size: 11px;
+}
+
+.share-dialog-msg__audience {
+  font-style: italic;
+}
+
 .annotation-collapsed-replies {
   display: none;
 }

diff --git a/h/templates/client/annotation.html b/h/templates/client/annotation.html
@@ -221,6 +221,18 @@
               class="share-dialog-target__icon h-icon-mail"></a>
           </span>
           <input class="share-dialog-link" type="text" value="{{vm.annotationURI}}" readonly>
+          <span class="share-dialog-msg" ng-if="vm.group() && !vm.group().public && !vm.isPrivate">
+            <span class="share-dialog-msg__audience">
+              Group.
+            </span>
+            Only group members will be able to view this annotation.
+          </span>
+          <span class="share-dialog-msg" ng-if="vm.isPrivate">
+            <span class="share-dialog-msg__audience">
+              Only me.
+            </span>
+            No one else will be able to view this annotation.
+          </span>
         </span>
       </span>
     </div>