diff --git a/app/helpers/artist_commentaries_helper.rb b/app/helpers/artist_commentaries_helper.rb new file mode 100644 index 000000000..c44ed7243 --- /dev/null +++ b/app/helpers/artist_commentaries_helper.rb @@ -0,0 +1,15 @@ +module ArtistCommentariesHelper + def format_commentary_title(title, classes: "") + tag.h3 do + tag.span(class: "prose #{classes}") do + format_text(title, disable_mentions: true, inline: true) + end + end + end + + def format_commentary_description(description, classes: "") + tag.div(class: "prose #{classes}") do + format_text(description, disable_mentions: true) + end + end +end diff --git a/app/javascript/src/styles/common/inline.scss b/app/javascript/src/styles/common/inline.scss index cd0efbdd7..a68b08f34 100644 --- a/app/javascript/src/styles/common/inline.scss +++ b/app/javascript/src/styles/common/inline.scss @@ -25,7 +25,7 @@ kbd.key { } /* used for "(deleted)" messages */ -span.inactive { +div.inactive, span.inactive { color: var(--muted-text-color); } diff --git a/app/views/artist_commentaries/_show.html.erb b/app/views/artist_commentaries/_show.html.erb index 3566ce84b..37c5774e0 100644 --- a/app/views/artist_commentaries/_show.html.erb +++ b/app/views/artist_commentaries/_show.html.erb @@ -13,28 +13,23 @@ <% if artist_commentary.original_present? %> <%= tag.section id: "original-artist-commentary", style: ("display: none" if artist_commentary.translated_present?) do %> -

<%= artist_commentary.original_title %>

-
- <%= format_text(artist_commentary.original_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(artist_commentary.original_title) %> + <%= format_commentary_description(artist_commentary.original_description) %> <% end %> <% end %> <% if artist_commentary.translated_present? %>
-

- <% if artist_commentary.translated_title.present? %> - <%= artist_commentary.translated_title %> - <% else %> - <%= artist_commentary.original_title %> - <% end %> -

-
- <% if artist_commentary.translated_description.present? %> - <%= format_text(artist_commentary.translated_description, :disable_mentions => true) %> - <% else %> - <%= format_text(artist_commentary.original_description, :disable_mentions => true) %> - <% end %> -
+ <% if artist_commentary.translated_title.present? %> + <%= format_commentary_title(artist_commentary.translated_title) %> + <% else %> + <%= format_commentary_title(artist_commentary.original_title, classes: "inactive") %> + <% end %> + + <% if artist_commentary.translated_description.present? %> + <%= format_commentary_description(artist_commentary.translated_description) %> + <% else %> + <%= format_commentary_description(artist_commentary.original_description, classes: "inactive") %> + <% end %>
<% end %> diff --git a/app/views/artist_commentaries/index.html.erb b/app/views/artist_commentaries/index.html.erb index 1057abdab..81fbd8ff9 100644 --- a/app/views/artist_commentaries/index.html.erb +++ b/app/views/artist_commentaries/index.html.erb @@ -17,16 +17,12 @@ <%= PostPresenter.preview(commentary.post, :tags => "status:any") %> -

<%= h(commentary.original_title) %>

-
- <%= format_text(commentary.original_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary.original_title) %> + <%= format_commentary_description(commentary.original_description) %> -

<%= h(commentary.translated_title) %>

-
- <%= format_text(commentary.translated_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary.translated_title) %> + <%= format_commentary_description(commentary.translated_description) %> <% end %> diff --git a/app/views/artist_commentary_versions/_revert_listing.html.erb b/app/views/artist_commentary_versions/_revert_listing.html.erb index 226cac309..32570bffa 100644 --- a/app/views/artist_commentary_versions/_revert_listing.html.erb +++ b/app/views/artist_commentary_versions/_revert_listing.html.erb @@ -20,16 +20,12 @@ <%= link_to commentary_version.post_id, post_path(commentary_version.post_id) %> -

<%= h(commentary_version.original_title) %>

-
- <%= format_text(commentary_version.original_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary_version.original_title) %> + <%= format_commentary_description(commentary_version.original_description) %> -

<%= h(commentary_version.translated_title) %>

-
- <%= format_text(commentary_version.translated_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary_version.translated_title) %> + <%= format_commentary_description(commentary_version.translated_description) %> <% if CurrentUser.is_moderator? %> diff --git a/app/views/artist_commentary_versions/_standard_listing.html.erb b/app/views/artist_commentary_versions/_standard_listing.html.erb index 08c0b4575..f9bc5824c 100644 --- a/app/views/artist_commentary_versions/_standard_listing.html.erb +++ b/app/views/artist_commentary_versions/_standard_listing.html.erb @@ -19,16 +19,12 @@ <%= PostPresenter.preview(commentary_version.post, :tags => "status:any") %> <%= link_to "#{commentary_version.post_id}.#{commentary_version.id}ยป", artist_commentary_versions_path(search: {post_id: commentary_version.post_id}) %> -

<%= h(commentary_version.original_title) %>

-
- <%= format_text(commentary_version.original_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary_version.original_title) %> + <%= format_commentary_description(commentary_version.original_description) %> -

<%= h(commentary_version.translated_title) %>

-
- <%= format_text(commentary_version.translated_description, :disable_mentions => true) %> -
+ <%= format_commentary_title(commentary_version.translated_title) %> + <%= format_commentary_description(commentary_version.translated_description) %> <% if CurrentUser.is_moderator? %>