UX: Show tags on flags too

This commit is contained in:
Robin Ward 2019-04-05 14:22:27 -04:00
parent 8181ad646b
commit a6f283da30
6 changed files with 42 additions and 34 deletions

View File

@ -1,11 +1,12 @@
{{reviewable-topic-link reviewable=reviewable tagName=''}}
<div class='post-contents-wrapper'>
{{reviewable-created-by user=reviewable.target_created_by tagName=''}}
<div class='post-contents'>
{{reviewable-created-by-name user=reviewable.target_created_by tagName=''}}
<div class='post-body'>
{{{reviewable.cooked}}}
{{reviewable-created-by user=reviewable.target_created_by tagName=''}}
<div class='post-contents'>
{{reviewable-created-by-name user=reviewable.target_created_by tagName=''}}
<div class='post-body'>
{{{reviewable.cooked}}}
</div>
{{yield}}
</div>
{{yield}}
</div>
</div>

View File

@ -10,11 +10,11 @@
</span>
<span class='status'>
{{#if reviewable.approved}}
<span class="approved"> {{d-icon "check"}} {{i18n "review.statuses.approved.title"}} </span>
<span class="approved"> {{d-icon "check"}} {{i18n "review.statuses.approved.title"}} </span>
{{else if reviewable.rejected}}
<span class="rejected"> {{d-icon "times"}} {{i18n "review.statuses.rejected.title"}} </span>
<span class="rejected"> {{d-icon "times"}} {{i18n "review.statuses.rejected.title"}} </span>
{{else if reviewable.ignored}}
<span class="ignored"> {{d-icon "external-link-alt"}} {{i18n "review.statuses.ignored.title"}} </span>
<span class="ignored"> {{d-icon "external-link-alt"}} {{i18n "review.statuses.ignored.title"}} </span>
{{/if}}
</span>
</div>
@ -36,10 +36,10 @@
</div>
{{else}}
{{#component reviewableComponent reviewable=reviewable tagName=''}}
<div class='reviewable-scores-and-history'>
{{reviewable-scores scores=reviewable.reviewable_scores tagName=''}}
{{reviewable-histories histories=reviewable.reviewable_histories tagName=''}}
</div>
<div class='reviewable-scores-and-history'>
{{reviewable-scores scores=reviewable.reviewable_scores tagName=''}}
{{reviewable-histories histories=reviewable.reviewable_histories tagName=''}}
</div>
{{/component}}
{{/if}}
</div>

View File

@ -1,29 +1,21 @@
{{#reviewable-topic-link reviewable=reviewable tagName=''}}
<div class="title-text">{{i18n "review.new_topic"}}
{{#reviewable-topic-link reviewable=reviewable tagName=''}}
<div class="title-text">{{i18n "review.new_topic"}}
{{reviewable.payload.title}}
</div>
{{category-badge reviewable.category}}
{{#if reviewable.payload.tags}}
<div class="list-tags">
{{#each reviewable.payload.tags as |t|}}
{{discourse-tag t}}
{{/each}}
</div>
{{/if}}
{{/reviewable-topic-link}}
{{reviewable-tags tags=reviewable.payload.tags tagName=''}}
{{/reviewable-topic-link}}
<div class='post-contents-wrapper'>
{{reviewable-created-by user=reviewable.created_by tagName=''}}
<div class='post-contents'>
{{reviewable-created-by-name user=reviewable.created_by tagName=''}}
<div class='post-contents'>
{{reviewable-created-by-name user=reviewable.created_by tagName=''}}
<div class='post-body'>
{{cook-text reviewable.payload.raw}}
<div class='post-body'>
{{cook-text reviewable.payload.raw}}
</div>
{{yield}}
</div>
{{yield}}
</div>
</div>

View File

@ -0,0 +1,5 @@
{{#if tags}}
<div class="list-tags">
{{#each tags as |t|}} {{discourse-tag t}} {{/each}}
</div>
{{/if}}

View File

@ -3,6 +3,7 @@
{{topic-status topic=reviewable.topic}}
<a href={{reviewable.topic_url}} class='title-text'>{{reviewable.topic.title}}</a>
{{category-badge reviewable.category}}
{{reviewable-tags tags=reviewable.topic_tags tagName=''}}
{{else if (has-block)}}
{{yield}}
{{else}}

View File

@ -11,11 +11,12 @@ class ReviewableSerializer < ApplicationSerializer
:type,
:topic_id,
:topic_url,
:topic_tags,
:category_id,
:created_at,
:can_edit,
:score,
:version
:version,
)
has_one :created_by, serializer: BasicUserSerializer, root: 'users'
@ -87,6 +88,14 @@ class ReviewableSerializer < ApplicationSerializer
data
end
def topic_tags
object.topic.tags.map(&:name)
end
def include_topic_tags?
object.topic.present? && SiteSetting.tagging_enabled?
end
def topic_url
return object.target.url if object.target.is_a?(Post)
return object.topic.url