From 83878e9c108934a4eb5ae4335075cbe55336cbcf Mon Sep 17 00:00:00 2001
From: Ben Halpern <bendhalpern@gmail.com>
Date: Wed, 6 Mar 2024 12:44:47 -0500
Subject: [PATCH] Add classes to content wrapper for articles and pages
 (#20731)

* Add classes to content wrapper for articles and pages

* Add class name check

* Fix org check
---
 app/assets/stylesheets/views/article.scss |  3 +--
 app/helpers/application_helper.rb         | 29 +++++++++++++++++------
 spec/requests/pages_spec.rb               |  3 ++-
 spec/requests/stories_show_spec.rb        |  7 ++++++
 4 files changed, 32 insertions(+), 10 deletions(-)

diff --git a/app/assets/stylesheets/views/article.scss b/app/assets/stylesheets/views/article.scss
index 7c3c4cf7a..f23105ab9 100644
--- a/app/assets/stylesheets/views/article.scss
+++ b/app/assets/stylesheets/views/article.scss
@@ -107,8 +107,7 @@
   z-index: var(--z-sticky);
 
   @media (min-width: $breakpoint-m) {
-    border-top-left-radius: 0;
-    border-top-right-radius: 0;
+    border-radius: var(--radius-large);
     padding: 0;
     padding-bottom: 0;
     background: var(--body-bg);
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index d047ffe60..2a28d1a37 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -67,14 +67,29 @@ module ApplicationHelper
   #
   # rubocop:disable Rails/HelperInstanceVariable
   def view_class
-    if @podcast_episode_show # custom due to edge cases
-      "stories stories-show podcast_episodes-show"
-    elsif @story_show
-      "stories stories-show"
-    else
-      "#{controller_name} #{current_page}"
-    end
+    base_classes = if @podcast_episode_show # custom due to edge cases
+                     "stories stories-show podcast_episodes-show"
+                   elsif @story_show
+                     "stories stories-show"
+                   else
+                     "#{controller_name} #{current_page}"
+                   end
+    base_classes += article_view_classes if @article&.class&.name&.start_with?("Article") # Article or ArticleDecorator
+    base_classes += page_view_classes if @page&.class&.name&.start_with?("Page")
+    base_classes
   end
+
+  def article_view_classes
+    base_classes = " #{@article.decorate.cached_tag_list_array.map { |tag| "articletag-#{tag}" }.join(' ')}"
+    base_classes += " articleuser-#{@article.user_id}"
+    base_classes += " articleorg-#{@article.organization_id}" if @article.organization_id
+    base_classes
+  end
+
+  def page_view_classes
+    " pageslug-#{@page.slug.gsub('/', '__SLASH__')}"
+  end
+
   # rubocop:enable Rails/HelperInstanceVariable
 
   # This function derives the appropriate "title" given the page_title.  Further it assigns the
diff --git a/spec/requests/pages_spec.rb b/spec/requests/pages_spec.rb
index e8a48bb65..bbb9f4467 100644
--- a/spec/requests/pages_spec.rb
+++ b/spec/requests/pages_spec.rb
@@ -9,12 +9,13 @@ RSpec.describe "Pages" do
       expect(response.body).to include("/page/#{page.slug}")
     end
 
-    it "has proper headline for top-level" do
+    it "has proper headline and classes for top-level" do
       page = create(:page, title: "Edna O'Brien96", is_top_level_path: true)
       get "/#{page.slug}"
       expect(response.body).to include(CGI.escapeHTML(page.title))
       expect(response.body).not_to include("/page/#{page.slug}")
       expect(response.body).to include("stories-show")
+      expect(response.body).to include(" pageslug-#{page.slug}")
     end
 
     context "when json template" do
diff --git a/spec/requests/stories_show_spec.rb b/spec/requests/stories_show_spec.rb
index aca0aa91c..e7d3fbd1e 100644
--- a/spec/requests/stories_show_spec.rb
+++ b/spec/requests/stories_show_spec.rb
@@ -100,6 +100,13 @@ RSpec.describe "StoriesShow" do
       expect(response.body).not_to include("<span class=\"fs-xl color-base-70 block\">Hey this is a test</span>")
     end
 
+    it "renders proper wrapper content clases" do
+      get article.path
+      expect(response.body)
+        .to include(" #{article.decorate.cached_tag_list_array.map { |tag| "articletag-#{tag}" }.join(' ')}")
+      expect(response.body).to include(" articleuser-#{article.user_id}")
+    end
+
     ###
 
     it "renders date-no-year if article published this year" do