From 1fa049a18ef5c7a8b4c2b2b8210624d02c2fe7e8 Mon Sep 17 00:00:00 2001
From: evazion <noizave@gmail.com>
Date: Tue, 16 Feb 2021 07:23:45 -0600
Subject: [PATCH] related tags: remove artist urls; move artist tag to
 translated tags.

* Remove artist URLs from related tags.
* Move the artist tag to the top of the translated tags list.
---
 .../src/styles/specific/related_tags.scss     |  4 ---
 app/views/related_tags/_source_tags.html.erb  | 34 +++----------------
 2 files changed, 5 insertions(+), 33 deletions(-)

diff --git a/app/javascript/src/styles/specific/related_tags.scss b/app/javascript/src/styles/specific/related_tags.scss
index bd201bebb..52eadd61c 100644
--- a/app/javascript/src/styles/specific/related_tags.scss
+++ b/app/javascript/src/styles/specific/related_tags.scss
@@ -14,10 +14,6 @@ div.related-tags {
   div.tag-column {
     max-width: 15em;
 
-    &.wide-column {
-      max-width: 45em;
-    }
-
     a.selected {
       background: var(--selected-related-tag-background-color);
       color: var(--selected-related-tag-color);
diff --git a/app/views/related_tags/_source_tags.html.erb b/app/views/related_tags/_source_tags.html.erb
index 961023b0e..2ebf962fa 100644
--- a/app/views/related_tags/_source_tags.html.erb
+++ b/app/views/related_tags/_source_tags.html.erb
@@ -1,33 +1,9 @@
 <div class="source-related-tags-columns flex space-x-4">
   <% if source.present? %>
-    <%= render "related_tags/tag_column", tags: source.translated_tags.map(&:name), class: "translated-tags-related-tags-column", title: "Translated Tags" %>
-
-    <div class="tag-column wide-column artist-related-tags-column">
-      <h3>Artist</h3>
-
-      <% if source.artists.size == 0 %>
-        none
-      <% else %>
-        <% if source.artists.any?(&:is_banned?) %>
-          <%= render_simple_tag_list(["banned_artist"]) %>
-        <% end %>
-
-        <% source.artists.each do |artist| %>
-          <%= render_simple_tag_list([artist.name]) %>
-
-          <ul>
-            <% artist.sorted_urls.each do |url| %>
-              <li>
-                <% if url.is_active %>
-                  <%= external_link_to url.url, truncate: 64, strip: :scheme, target: "_blank" %>
-                <% else %>
-                  <del><%= external_link_to url.url, truncate: 64, strip: :scheme, target: "_blank" %></del>
-                <% end %>
-              </li>
-            <% end %>
-          </ul>
-        <% end %>
-      <% end %>
-    </div>
+    <%# XXX %>
+    <% tags = source.artists.map(&:name) %>
+    <% tags += ["banned_artist"] if source.artists.any?(&:is_banned?) %>
+    <% tags += source.translated_tags.map(&:name) %>
+    <%= render "related_tags/tag_column", tags: tags, class: "translated-tags-related-tags-column", title: "Translated Tags" %>
   <% end %>
 </div>