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.
This commit is contained in:
evazion
2021-02-16 07:23:45 -06:00
parent 19aea560db
commit 1fa049a18e
2 changed files with 5 additions and 33 deletions

View File

@@ -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);

View File

@@ -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>