thredded-workgroup 0.2.0 → 0.3.1

data/app/assets/stylesheets/thredded/workgroup/_topics.scss

@@ -1,6 +1,7 @@
 .thredded--topic-source cite a {
   color: #a5aab6;
   text-decoration: none;
+
   &:hover {
     color: #4a90e2;
 
@@ -17,36 +18,205 @@ cite.thredded--messageboard-name {
 .thredded--topic-post-and-last-user {
   max-width: 35rem;
   margin: auto;
-  margin-top: 1rem;
+  padding-top: 1rem;
+}
+
+.thredded--topics--topic {
+  a.thredded-follow-toggle {
+    position: absolute;
+    right: -1.6rem;
+    top: 0;
+
+    svg, svg.thredded--topics--follow-icon {
+      position: static;
+      fill: currentColor;
+      display: inline-block;
+      font-size: 1em;
+      width: 1.4rem;
+      height: 1.4rem;
+      opacity: 0.4;
+    }
 
-  &.thredded--condensed {
-    max-height: 100px;
-    overflow: hidden;
-    opacity: 0.8;
-    &.thredded--hovering {
-      opacity: 1;
+    &:hover svg {
+      cursor: pointer;
+      opacity: 0.7;
+      color: #4a90e2;
     }
   }
-  &.thredded--hovering {
-    cursor: pointer;
+
+  &.thredded--topic-following a.thredded-follow-toggle {
+    svg.thredded--topics--unfollow-icon {
+      display: none;
+    }
+
+    &:hover, &.thredded--topic-hovering {
+      svg.thredded--topics--follow-icon {
+        color: darken($thredded-brand, 20%);
+      }
+    }
   }
-}
 
-.thredded--topics--topic .thredded--topics--follow-icon {
-  &:hover {
-    cursor: pointer;
-    opacity: 0.7;
-    color: #4a90e2;
+  &.thredded--topic-notfollowing a.thredded-follow-toggle {
+    svg.thredded--topics--unfollow-icon {
+      opacity: 0.2;
+    }
+
+    svg.thredded--topics--follow-icon {
+      display: none;
+    }
+
+    &:hover, &.thredded--topic-hovering {
+      svg.thredded--topics--unfollow-icon {
+        opacity: 0.5;
+      }
+    }
   }
 }
 
-.thredded--topics--topic.thredded--rollover{
+
+.thredded--topics--topic.thredded--topic-hovering {
   cursor: pointer;
 
   .thredded--topic-title a {
     color: #4a90e2;
+
     &:hover {
       cursor: pointer;
     }
   }
+
+  .thredded--topic-controls {
+    .thredded--topic--view-button {
+      background-color: $thredded-button-hover-background;
+      color: $thredded-button-hover-color;
+    }
+  }
+}
+
+.thredded--topic-controls {
+  .thredded--topic--read-state-toggle.thredded--topic-hovering {
+    background-color: $thredded-button-hover-background;
+    color: $thredded-button-hover-color;
+  }
+}
+
+.thredded--topic-last-post-with-controls {
+  position: relative;
+  top: 0;
+}
+
+.thredded--topic-controls {
+  display: none;
+  position: absolute;
+  top: 5px;
+  right: 0;
+  z-index: 4;
+
+  .thredded--topic-hovering & {
+    display: block;
+    background: #ffffff88;
+  }
+
+  .thredded--icon {
+    height: 0.85em;
+    margin: 0;
+  }
+}
+
+.thredded--topics--topic > .thredded--topic--read-state-toggle {
+  border-radius: 50%;
+  display: inline-block;
+  font-weight: 900;
+  font-size: 0.8rem;
+  height: 2rem;
+  left: -$thredded-topics-topic-posts-counter-width;
+  line-height: 2rem;
+  margin-right: $thredded-base-spacing;
+  position: absolute;
+  text-align: center;
+  top: 0;
+  width: 2rem;
+  transition: background 0.1s linear, color 0.1s linear;
+
+  .thredded--icon {
+    height: 1.25em;
+    position: relative;
+    top: 10%;
+  }
+}
+
+.thredded--topic-read {
+  > .thredded--topics--title a {
+    font-weight: lighter;
+  }
+
+  > .thredded--topic--read-state-toggle {
+    background: $thredded-badge-inactive-background;
+    color: $thredded-badge-inactive-color;
+
+    &.thredded--topic-hovering {
+      background: darken(opacify($thredded-badge-inactive-background, 0.25), 20%);
+    }
+  }
+
+  .thredded--topic--read-state-toggle.thredded--topic--mark-as-read {
+    display: none
+  }
+
+}
+
+.thredded--topic-unread {
+  > .thredded--topics--title a {
+    font-weight: bold;
+  }
+
+  > .thredded--topic--read-state-toggle {
+    background: $thredded-badge-active-background;
+    color: $thredded-badge-active-color;
+
+    .thredded--topics--read-icon {
+      display: none;
+    }
+
+    &.thredded--topic-hovering {
+      background: darken($thredded-badge-active-background, 20%);
+    }
+  }
+
+  .thredded--topic--read-state-toggle.thredded--topic--mark-as-unread {
+    display: none
+  }
+}
+
+.thredded--button {
+  svg.thredded--icon {
+    max-width: 16px;
+  }
+
+  &.thredded--topic-hovering {
+    background: darken($thredded-badge-active-background, 20%);
+  }
+}
+
+.thredded--topic-following {
+  .thredded--button.thredded--topic--follow {
+    display: none;
+  }
+}
+
+.thredded--topic-notfollowing {
+  .thredded--button.thredded--topic--unfollow {
+    display: none;
+  }
+}
+
+@include thredded-media-mobile {
+  .thredded--topic-hovering {
+    .thredded--button.thredded--topic--mark-as-unread,
+    .thredded--button.thredded--topic--mark-as-read,
+    .thredded--button.thredded--topic--follow,
+    .thredded--button.thredded--topic--unfollow {
+      display: none;
+    }
+  }
 }

data/app/assets/stylesheets/thredded-workgroup.scss

@@ -1,3 +1,5 @@
 @import 'thredded';
+@import 'thredded/workgroup/followers';
+@import 'thredded/workgroup/overflow';
 @import 'thredded/workgroup/navs';
 @import 'thredded/workgroup/topics';

data/app/controllers/concerns/thredded/workgroup/all_unread_followed_topics.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Thredded
+  module Workgroup
+    module AllUnreadFollowedTopics
+      extend ActiveSupport::Concern
+      included do
+        if defined?(Pundit::Authorization)
+          include Pundit::Authorization
+        else
+          include Pundit
+        end
+        helper_method :all_unread_followed_topics_count
+      end
+
+      # @return [ActiveRecord::Relation]
+      def all_topics_scope
+        @all_topics_scope ||=
+          policy_scope(Thredded::Topic.all).joins(:messageboard).merge(policy_scope(Thredded::Messageboard.all))
+      end
+
+      # @return Integer
+      def all_unread_followed_topics_count
+        @all_unread_followed_topics_count ||=
+          if thredded_signed_in?
+            all_topics_scope.unread_followed_by(thredded_current_user).count
+          else
+            0
+          end
+      end
+
+      # need to define `thredded_current_user` and possibly `pundit_user`
+    end
+  end
+end

data/app/controllers/thredded/application_controller.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require_dependency File.expand_path("../../app/controllers/thredded/application_controller",
+                                    Thredded::Engine.called_from)
+
+module Thredded
+  module ApplicationControllerRetaining
+    def topics_scope
+      return super if messageboard_or_nil
+      all_topics_scope
+    end
+
+    def unread_followed_topics_count
+      return super if messageboard_or_nil
+      all_unread_followed_topics_count
+    end
+  end
+  class ApplicationController
+    include Thredded::Workgroup::AllUnreadFollowedTopics
+
+    prepend ::Thredded::ApplicationControllerRetaining
+  end
+end

data/app/controllers/thredded/posts_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_dependency File.expand_path("../../app/controllers/thredded/posts_controller", Thredded::Engine.called_from)
 
 module Thredded
@@ -9,11 +10,11 @@ module Thredded
       )
       authorize_creating @post_form.post
 
-      if @post_form.save # rubocop:disable Style/GuardClause
+      if @post_form.save
         # TODO: extract as a hook on thredded#posts_controller `after_create(post)`
         redirect_after_create(@post_form.post)
       else
-        return render :new
+        render :new
       end
     end
 

data/app/controllers/thredded/workgroup/application_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Thredded
   module Workgroup
     class ApplicationController < Thredded::ApplicationController

data/app/controllers/thredded/workgroup/navs_controller.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Thredded
   module Workgroup
     class NavsController < Thredded::Workgroup::ApplicationController

data/app/controllers/thredded/workgroup/read_states_controller.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Thredded
+  module Workgroup
+    class ReadStatesController < ::Thredded::Workgroup::ApplicationController
+      before_action :thredded_require_login!
+
+      def mark_all_topics_read
+        ::Thredded::Workgroup::MarkAllTopicsReadJob.perform_later(thredded_current_user) if thredded_signed_in?
+        redirect_to request.referer, flash: { notice: "Marking all topics as read (will take a few seconds)" }
+      end
+    end
+  end
+end

data/app/controllers/thredded/workgroup/topics_controller.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+# frozen_string_literal: true # frozen_string_literal: true
+
+module Thredded
+  module Workgroup
+    class TopicsController < Thredded::Workgroup::ApplicationController
+      before_action :thredded_require_login!
+
+      def kick
+        if (user_topic_follow = UserTopicFollow.where(topic_id: params[:id], user_id: params[:user_id]).first)
+          user_topic_follow.destroy
+        end
+        render plain: "Ok"
+      end
+    end
+  end
+end

data/app/helpers/thredded/application_helper.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_dependency File.expand_path("../../app/helpers/thredded/application_helper", Thredded::Engine.called_from)
 module Thredded
   module ApplicationHelper

data/app/helpers/thredded/workgroup/application_helper.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module Thredded
   module Workgroup
     module ApplicationHelper

data/app/jobs/thredded/workgroup/mark_all_topics_read_job.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Thredded
+  module Workgroup
+    class MarkAllTopicsReadJob < ::ActiveJob::Base
+      def perform(user)
+        ::Thredded::Topic.unread(user).each do |topic|
+          ::Thredded::UserTopicReadState.touch!(user.id, topic.last_post)
+        end
+      end
+    end
+  end
+end

data/app/view_hooks/thredded/all_view_hooks.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+require_dependency File.expand_path("../../app/view_hooks/thredded/all_view_hooks", Thredded::Engine.called_from)
+
+module Thredded
+  class AllViewHooks
+    prepend ::Thredded::Workgroup::ViewHooks
+  end
+end

data/app/view_hooks/thredded/workgroup/view_hooks.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Thredded
+  module Workgroup
+    module ViewHooks
+      attr_reader :topic_with_last_post
+
+      def initialize
+        super
+        @topic_with_last_post = TopicWithLastPost.new
+      end
+
+      # View hooks for a Topic when displaying last post.
+      class TopicWithLastPost
+        # @return [Thredded::AllViewHooks::ViewHook]
+        attr_reader :last_post_with_controls
+
+        # @return [Thredded::AllViewHooks::ViewHook]
+        attr_reader :last_post
+
+        def initialize
+          @last_post_with_controls = Thredded::AllViewHooks::ViewHook.new
+          @last_post = Thredded::AllViewHooks::ViewHook.new
+        end
+      end
+    end
+  end
+end

data/app/view_models/thredded/topic_view.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_dependency File.expand_path("../../app/view_models/thredded/topic_view", Thredded::Engine.called_from)
 module Thredded
   class TopicView < Thredded::BaseTopicView
@@ -11,5 +12,21 @@ module Thredded
       page = @read_state.first_unread_post_page || @read_state.last_read_post_page
       Thredded::UrlsHelper.topic_path(@topic, page: page, anchor: anchor)
     end
+
+    def reply_path
+      Thredded::UrlsHelper.topic_path(@topic, page: last_page, anchor: "new_post")
+    end
+
+    def mark_as_unread_path
+      Thredded::UrlsHelper.mark_as_unread_post_path(id: last_post.id, format: :json)
+    end
+
+    def mark_as_read_path
+      Thredded::UrlsHelper.mark_as_read_post_path(id: last_post.id, format: :json)
+    end
+
+    def last_page
+      (@topic.posts_count / Thredded.posts_per_page.to_f).ceil # TODO: doesn't account for visibility
+    end
   end
 end

data/app/view_models/thredded/topics_page_view.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_dependency File.expand_path("../../app/view_models/thredded/topics_page_view", Thredded::Engine.called_from)
 module Thredded
   module TopicsPageViewWithLastPost

data/app/views/thredded/shared/nav/_unread_topics.html.erb

@@ -0,0 +1 @@
+<% # overriding thredded's view  %>

data/app/views/thredded/topics/_followers.html.erb

@@ -0,0 +1,23 @@
+<% if Thredded.show_topic_followers %>
+  <%= define_svg_icons "thredded/follow.svg", "thredded/unfollow.svg" %>
+  <div class="thredded--topic-followers">
+    <% if topic.followers.length > 0 %>
+      <%= t('thredded.topics.followed_by')%>
+      <% topic.followers.each do |user| %>
+        <span class="thredded--follower user-<%= user.id %>">
+          <%= user_mention(user) %>
+          <%- if user == thredded_current_user %>
+            <%= t("thredded.workgroup.following.you")%>
+          <%- else %>
+            <button class="thredded--follower--remove" data-kick-path="<%= thredded_workgroup.kick_topic_path(topic.to_model.id, user_id: user.id) %>" title="Remove follower" aria-label="Remove follower">
+              <%= shared_inline_svg "thredded/follow.svg", class: "thredded--topics--follow-icon", role: "img" %>
+              <%= shared_inline_svg "thredded/unfollow.svg", class: "thredded--topics--unfollow-icon", role: "img" %>
+            </button>
+          <%- end %>
+        </span>
+      <% end %>
+    <% else %>
+      <%= t('thredded.topics.followed_by_noone')%>
+    <% end %>
+  </div>
+<% end %>

data/app/views/thredded/topics/_topic.html.erb

@@ -2,22 +2,18 @@
     id:    dom_id(topic),
     class: ['thredded--topics--topic', topic_css_classes(topic)],
     data: {topic: topic.id, messageboard: topic.messageboard_id} do %>
-  <div class="thredded--topics--posts-count"><%= topic.posts_count %></div>
+
+  <div class="thredded--topic--read-state-toggle">
+    <%= shared_inline_svg("thredded/workgroup/envelope-open-o.svg", class: "thredded--topics--read-icon thredded--icon", role: "img", viewBox: "0 0 1792 1792",  title: "Mark as read") %>
+    <%= shared_inline_svg("thredded/workgroup/envelope-o.svg", class: "thredded--topics--unread-icon thredded--icon", role: "img", viewBox: "0 0 1792 1792", title: "Mark as unread") %>
+  </div>
 
   <h1 class="thredded--topics--title">
     <span class="thredded--topic-title"><%= link_to topic.title, topic.path %></span>
-    <% unless messageboard_or_nil #don't show if being viewed in the context of a messageboard%>
-      <cite class="thredded--messageboard-name">
-        <%= link_to topic.messageboard_name, topic.messageboard_path %>
-      </cite>
-    <% end %>
-    <svg viewBox="0 0 116 121" class="thredded--topics--follow-icon">
-      <% if topic.followed? %>
-        <use xlink:href="#thredded-follow-icon"/>
-      <% else %>
-        <use xlink:href="#thredded-unfollow-icon"/>
-      <% end %>
-    </svg>
+    <a href="#" class="thredded-follow-toggle js-thredded-follow-toggle">
+      <%= shared_inline_svg "thredded/follow.svg", class: "thredded--topics--follow-icon", role: "img" %>
+      <%= shared_inline_svg "thredded/unfollow.svg", class: "thredded--topics--unfollow-icon", role: "img" %>
+    </a>
   </h1>
 
   <% if topic.categories.any? %>
@@ -26,26 +22,27 @@
     </ul>
   <% end %>
 
+  <cite class="thredded--topics--updated-by">
+    <%= time_ago topic.last_post_at %>
+    <%= user_link topic.last_user %>
+  </cite>
+
+  <% if local_assigns[:show_messageboard] %>
+    <span class="thredded--topics--messageboard">
+      <%= t 'thredded.topics.in_messageboard_html',
+            messageboard_link: link_to(topic.messageboard_name, topic.messageboard_path) %>
+    </span>
+  <% end %>
 
   <% if topic.blocked? && topic.can_moderate? %>
     <span class="thredded--topics--moderation-state thredded--alert thredded--alert-danger">
       <%= render 'thredded/shared/content_moderation_blocked_state', moderation_record: topic.last_moderation_record %>
     </span>
   <% end %>
+  <%= render 'thredded/workgroup/topics/last_post_with_controls', topic: topic if topic.last_post %>
+<% end %>
 
-  <%= render partial: 'thredded/topics/followers', locals: {topic: topic} %>
-
-  <div class='thredded--topic-post-and-last-user thredded--condensed'>
-    <div class="thredded--topic-source">
-      <cite class="thredded--topics--updated-by">
-        <%= topic.last_user.thredded_display_name %>
-        <%= time_ago topic.last_post_at %>
-      </cite>
-    </div>
-
-    <div class="thredded--last-post">
-      <%= render 'thredded/posts_common/content', post: topic.last_post if topic.last_post %>
-    </div>
-  </div>
-
+<% if local_assigns[:sticky_topics_divider] &&
+    !topic_iteration.last? && topic.sticky? && !topics[topic_counter + 1].sticky? %>
+  <%= render 'thredded/topics/sticky_topics_divider' %>
 <% end %>

data/app/views/thredded/workgroup/navs/_personal_nav.html.erb

@@ -1,6 +1,12 @@
 <ul class="thredded--workgroup">
   <li class="<%= Thredded::Workgroup.navbar_class(params, :unread) %>">
-    <%= link_to t('thredded.workgroup.unread.link'), thredded_workgroup.unread_nav_path%>
+    <%= link_to thredded_workgroup.unread_nav_path do%>
+      <%= t('thredded.workgroup.unread.link') %>
+      <% if all_unread_followed_topics_count > 0 -%>
+      <span class="thredded--user-navigation--unread-topics--followed-count"><%=shared_inline_svg "thredded/follow.svg", class: "thredded--unread-topics--followed-icon", role:"img" %>
+        <%= all_unread_followed_topics_count %></span>
+      <% end -%>
+    <% end -%>
   </li>
   <li class="<%= Thredded::Workgroup.navbar_class(params, :awaiting) %>">
     <%= link_to thredded_workgroup.awaiting_nav_path do %>

data/app/views/thredded/workgroup/navs/all_topics.html.erb

@@ -6,4 +6,4 @@
   </ul>
 <% end %>
 
-<%= render partial: 'thredded/topics/topics_with_last_post'%>
+<%= render partial: 'thredded/workgroup/topics/topics_with_last_post'%>

data/app/views/thredded/workgroup/navs/awaiting.html.erb

@@ -6,4 +6,4 @@
   </ul>
 <% end %>
 
-<%= render partial: 'thredded/topics/topics_with_last_post'%>
+<%= render partial: 'thredded/workgroup/topics/topics_with_last_post'%>

data/app/views/thredded/workgroup/navs/following.html.erb

@@ -6,4 +6,4 @@
   </ul>
 <% end %>
 
-<%= render partial: 'thredded/topics/topics_with_last_post'%>
+<%= render partial: 'thredded/workgroup/topics/topics_with_last_post'%>

data/app/views/thredded/workgroup/navs/unread.html.erb

@@ -6,4 +6,4 @@
   </ul>
 <% end %>
 
-<%= render partial: 'thredded/topics/topics_with_last_post'%>
+<%= render partial: 'thredded/workgroup/topics/topics_with_last_post'%>

data/app/views/thredded/workgroup/topics/_controls.html.erb

@@ -0,0 +1,21 @@
+<div class="thredded--topic-controls">
+  <a class="thredded--topic--read-state-toggle thredded--topic--mark-as-read thredded--button" data-post-path="<%= topic.mark_as_read_path %>">
+    <%= shared_inline_svg("thredded/workgroup/envelope-open-o.svg", class: "thredded--topics--read-icon thredded--icon", role: "img", viewBox: "0 0 1792 1792") %>
+    <%= t("thredded.topics.mark_read")%>
+  </a>
+  <a class="thredded--topic--read-state-toggle thredded--topic--mark-as-unread thredded--button" data-post-path="<%= topic.mark_as_unread_path %>">
+    <%= shared_inline_svg("thredded/workgroup/envelope-o.svg", class: "thredded--topics--unread-icon thredded--icon", role: "img", viewBox: "0 0 1792 1792") %>
+    <%= t("thredded.topics.mark_unread")%>
+  </a>
+  <a class="thredded--topic--view-button thredded--button" href="<%= topic.reply_path %>">
+    <%= t("thredded.topics.view_or_reply")%>
+  </a>
+  <a href="#" class="js-thredded-follow-toggle thredded--topic--follow thredded--button">
+    <%= shared_inline_svg "thredded/follow.svg", class: "thredded--icon", role: "img" %>
+    <%= t("thredded.topics.follow")%>
+  </a>
+  <a href="#" class="js-thredded-follow-toggle thredded--topic--unfollow thredded--button">
+    <%= shared_inline_svg "thredded/unfollow.svg", class: "thredded--icon", role: "img" %>
+    <%= t("thredded.topics.unfollow")%>
+  </a>
+</div>

data/app/views/thredded/workgroup/topics/_last_post.html.erb

@@ -0,0 +1,11 @@
+<%= view_hooks.topic_with_last_post.last_post.render self, topic: topic do %>
+  <div class="thredded--topic-source">
+    <cite class="thredded--topics--updated-by">
+      <%= topic.last_user.thredded_display_name %>
+      <%= time_ago topic.last_post_at %>
+    </cite>
+  </div>
+  <div class="thredded--last-post">
+    <%= render 'thredded/posts_common/content', post: topic.last_post %>
+  </div>
+<% end %>

data/app/views/thredded/workgroup/topics/_last_post_with_controls.html.erb

@@ -0,0 +1,12 @@
+<%= view_hooks.topic_with_last_post.last_post_with_controls.render self, topic: topic do %>
+  <div class="thredded--topic-last-post-with-controls">
+    <%= render "thredded/workgroup/topics/controls", topic: topic %>
+    <div class='thredded--topic-post-and-last-user thredded--condensable thredded--condensable--condensed'>
+      <%= render "thredded/workgroup/topics/last_post", topic: topic %>
+      <div class="thredded--condensable--overflow-only">
+        <span class="thredded--condensable--expand"><%= t("thredded.workgroup.condensable.read_all") %></span>
+        <span class="thredded--condensable--condense"><%= t("thredded.workgroup.condensable.read_less") %></span>
+      </div>
+    </div>
+  </div>
+<% end %>