From 536d1fe0e7f5bfea2fddb0726e7492ab7fd60fa7 Mon Sep 17 00:00:00 2001
From: Steffen van Bergerem <svbergerem@online.de>
Date: Thu, 30 Jul 2015 01:17:07 +0200
Subject: [PATCH] Move nsfw-shield to status-message

closes #6270
---
 Changelog.md                                  |  1 +
 app/assets/stylesheets/single-post-view.scss  |  2 +
 app/assets/stylesheets/stream_element.scss    |  3 ++
 .../templates/status-message_tpl.jst.hbs      | 28 ++++++++++++-
 .../templates/stream-element_tpl.jst.hbs      | 42 ++++---------------
 .../javascripts/app/views/stream_post_spec.js |  2 +-
 6 files changed, 40 insertions(+), 38 deletions(-)

diff --git a/Changelog.md b/Changelog.md
index 738f2a3d8..6666ad01c 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -16,6 +16,7 @@
 ## Features
 
 * Implement NodeInfo [#6239](https://github.com/diaspora/diaspora/pull/6239)
+* Display original author on reshares of NSFW posts [#6270](https://github.com/diaspora/diaspora/pull/6270)
 
 # 0.5.2.0
 
diff --git a/app/assets/stylesheets/single-post-view.scss b/app/assets/stylesheets/single-post-view.scss
index 7ddcb9576..d39f20d42 100644
--- a/app/assets/stylesheets/single-post-view.scss
+++ b/app/assets/stylesheets/single-post-view.scss
@@ -103,6 +103,8 @@
       padding-left: 10px;
     }
 
+    .nsfw-off { display: none; }
+    .nsfw-shield { display: none; }
     .oembed { width: 95%; }
     .photo_attachments {
       img.big_stream_photo { max-width: 90%; }
diff --git a/app/assets/stylesheets/stream_element.scss b/app/assets/stylesheets/stream_element.scss
index 0041de0da..3c3a946f8 100644
--- a/app/assets/stylesheets/stream_element.scss
+++ b/app/assets/stylesheets/stream_element.scss
@@ -45,6 +45,9 @@
   border-bottom: 1px solid $border-grey;
   padding: 10px;
   & > .media {
+    &.shield-active .nsfw-hidden { display: none; }
+    &:not(.shield-active) .nsfw-shield { display: none; }
+    &:not(.shield-off) .nsfw-off { display: none; }
     & > .img > .avatar {
       margin-top: 5px;
       &.small {
diff --git a/app/assets/templates/status-message_tpl.jst.hbs b/app/assets/templates/status-message_tpl.jst.hbs
index cf576b853..31a098277 100644
--- a/app/assets/templates/status-message_tpl.jst.hbs
+++ b/app/assets/templates/status-message_tpl.jst.hbs
@@ -1,5 +1,29 @@
+<div class="nsfw-shield">
+  <strong>
+    #NSFW
+  </strong>
+  |
+  <a href="#" class="show_nsfw_post">
+    {{t "stream.show_nsfw_post"}}
+  </a>
+  |
+  <a href="#" class="toggle_nsfw_state">
+    {{t "stream.show_nsfw_posts"}}
+  </a>
+</div>
+
+<div class="nsfw-off">
+  <strong>
+    #NSFW
+  </strong>
+  |
+  <a href="#" class="toggle_nsfw_state">
+      {{t "stream.hide_nsfw_posts"}}
+  </a>
+</div>
+
 {{#if largePhoto}}
-  <div class="photo_attachments">
+  <div class="photo_attachments nsfw-hidden">
     <a href="#" class="stream-photo-link">
       {{#with largePhoto}}
         <img src="{{sizes.large}}" class="stream-photo big_stream_photo" data-small-photo="{{sizes.small}}" data-full-photo="{{sizes.large}}" rel="lightbox">
@@ -14,7 +38,7 @@
   </div>
 {{/if}}
 
-<div class="collapsible">
+<div class="collapsible nsfw-hidden">
   <div class="markdown-content">
     {{#if text}}
       {{{text}}}
diff --git a/app/assets/templates/stream-element_tpl.jst.hbs b/app/assets/templates/stream-element_tpl.jst.hbs
index 6029e056b..3cd2e02c7 100644
--- a/app/assets/templates/stream-element_tpl.jst.hbs
+++ b/app/assets/templates/stream-element_tpl.jst.hbs
@@ -1,5 +1,4 @@
-<div class="media">
-
+<div class="media {{#if showPost}} {{#if nsfw}} shield-off {{/if}} {{else}} shield-active {{/if}}">
   {{#with author}}
     <a href="/people/{{guid}}" class="img {{{hovercardable this}}}">
       {{{personImage this}}}
@@ -54,38 +53,11 @@
       </span>
     </div>
 
-    {{#if showPost}}
-        {{#if nsfw}}
-          <div class="nsfw_off">
-            <strong>
-              #NSFW
-            </strong>
-            |
-            <a href="#" class="toggle_nsfw_state">
-                {{t "stream.hide_nsfw_posts"}}
-            </a>
-          </div>
-        {{/if}}
-      <div class="post-content"> </div>
-      <div class="status-message-location"> </div>
-
-      <div class="feedback"> </div>
-      <div class="likes"> </div>
-      <div class="comments"> </div>
-    {{else}}
-      <div class="nsfw-shield">
-        <strong>
-          #NSFW
-        </strong>
-        |
-        <a href="#" class="show_nsfw_post">
-          {{t "stream.show_nsfw_post"}}
-        </a>
-        |
-        <a href="#" class="toggle_nsfw_state">
-          {{t "stream.show_nsfw_posts"}}
-        </a>
-      </div>
-    {{/if}}
+    <div class="post-content"> </div>
+    <div class="status-message-location nsfw-hidden"> </div>
 
+    <div class="feedback nsfw-hidden"> </div>
+    <div class="likes nsfw-hidden"> </div>
+    <div class="comments nsfw-hidden"> </div>
   </div>
+</div>
diff --git a/spec/javascripts/app/views/stream_post_spec.js b/spec/javascripts/app/views/stream_post_spec.js
index e660db727..47084bd6e 100644
--- a/spec/javascripts/app/views/stream_post_spec.js
+++ b/spec/javascripts/app/views/stream_post_spec.js
@@ -144,7 +144,7 @@ describe("app.views.StreamPost", function(){
         this.view = new this.PostViewClass({model : this.statusMessage}).render();
 
         this.hiddenPosts = function(){
-          return this.view.$(".nsfw-shield");
+          return this.view.$(".media.shield-active .nsfw-shield");
         };
       });