UX: Tweaks to filtered replies (#11485)

Include the enable_filtered_replies_view site setting in the admin UI
Adds title label to in-reply-to widget
Invokes the filtered UI when using replies_to_post_number as a query
parameter
Replaces the "Show All" button icon
Fixes grammar for "Viewing 1 reply to..." label
This commit is contained in:
Penar Musaraj 2020-12-15 11:23:44 -05:00 committed by GitHub
parent 47fa3cf864
commit 31fe0e91ad
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 14 additions and 9 deletions

View File

@ -52,7 +52,7 @@ export default Controller.extend(bufferedProperty("model"), {
multiSelect: false, multiSelect: false,
selectedPostIds: null, selectedPostIds: null,
editingTopic: false, editingTopic: false,
queryParams: ["filter", "username_filters"], queryParams: ["filter", "username_filters", "replies_to_post_number"],
loadedAllPosts: or( loadedAllPosts: or(
"model.postStream.loadedAllPosts", "model.postStream.loadedAllPosts",
"model.postStream.loadingLastPost" "model.postStream.loadingLastPost"
@ -64,6 +64,7 @@ export default Controller.extend(bufferedProperty("model"), {
_progressIndex: null, _progressIndex: null,
hasScrolled: null, hasScrolled: null,
username_filters: null, username_filters: null,
replies_to_post_number: null,
filter: null, filter: null,
quoteState: null, quoteState: null,

View File

@ -46,7 +46,8 @@ export default RestModel.extend({
stream: [], stream: [],
userFilters: [], userFilters: [],
summary: false, summary: false,
filterRepliesToPostNumber: false, filterRepliesToPostNumber:
parseInt(this.get("topic.replies_to_post_number"), 10) || false,
filterUpwardsPostID: false, filterUpwardsPostID: false,
loaded: false, loaded: false,
loadingAbove: false, loadingAbove: false,

View File

@ -90,8 +90,8 @@ createWidget("posts-filtered-notice", {
return [ return [
h( h(
"span.filtered-replies-viewing", "span.filtered-replies-viewing",
I18n.t("post.filtered_replies.viewing", { I18n.t("post.filtered_replies_viewing", {
reply_count: sourcePost.reply_count, count: sourcePost.reply_count,
}) })
), ),
h("span.filtered-user-row", [ h("span.filtered-user-row", [
@ -170,7 +170,7 @@ createWidget("filter-show-all", {
}, },
html() { html() {
return [iconNode("far-comments"), I18n.t("post.filtered_replies.show_all")]; return [iconNode("arrows-alt-v"), I18n.t("post.filtered_replies.show_all")];
}, },
click() { click() {

View File

@ -120,7 +120,7 @@ createWidget("select-post", {
createWidget("reply-to-tab", { createWidget("reply-to-tab", {
tagName: "a.reply-to-tab", tagName: "a.reply-to-tab",
buildKey: (attrs) => `reply-to-tab-${attrs.id}`, buildKey: (attrs) => `reply-to-tab-${attrs.id}`,
title: "post.in_reply_to",
defaultState() { defaultState() {
return { loading: false }; return { loading: false };
}, },

View File

@ -2752,6 +2752,11 @@ en:
filtered_replies_hint: filtered_replies_hint:
one: "View this post and its reply" one: "View this post and its reply"
other: "View this post and its %{count} replies" other: "View this post and its %{count} replies"
filtered_replies_viewing:
one: "Viewing %{count} reply to"
other: "Viewing %{count} replies to"
in_reply_to: "Load parent post"
errors: errors:
create: "Sorry, there was an error creating your post. Please try again." create: "Sorry, there was an error creating your post. Please try again."
@ -2924,7 +2929,6 @@ en:
description: "Edit the bookmark name or change the reminder date and time" description: "Edit the bookmark name or change the reminder date and time"
filtered_replies: filtered_replies:
viewing: "Viewing %{reply_count} replies to"
viewing_posts_by: "Viewing %{post_count} posts by" viewing_posts_by: "Viewing %{post_count} posts by"
viewing_subset: "Some replies are collapsed" viewing_subset: "Some replies are collapsed"
viewing_summary: "Viewing a summary of this topic" viewing_summary: "Viewing a summary of this topic"

View File

@ -1863,7 +1863,7 @@ en:
body_min_entropy: "The minimum entropy (unique characters, non-english count for more) required for a post body." body_min_entropy: "The minimum entropy (unique characters, non-english count for more) required for a post body."
allow_uppercase_posts: "Allow all caps in a topic title or a post body." allow_uppercase_posts: "Allow all caps in a topic title or a post body."
max_consecutive_replies: "Number of posts a user has to make in a row in a topic before being prevented from adding another reply." max_consecutive_replies: "Number of posts a user has to make in a row in a topic before being prevented from adding another reply."
enable_filtered_replies_view: "(n) replies button should collapse all other posts and only show the selected replies." enable_filtered_replies_view: '"(n) replies" button collapses all other posts and only shows the current post and its replies.'
title_fancy_entities: "Convert common ASCII characters to fancy HTML entities in topic titles, ala SmartyPants <a href='https://daringfireball.net/projects/smartypants/' target='_blank'>https://daringfireball.net/projects/smartypants/</a>" title_fancy_entities: "Convert common ASCII characters to fancy HTML entities in topic titles, ala SmartyPants <a href='https://daringfireball.net/projects/smartypants/' target='_blank'>https://daringfireball.net/projects/smartypants/</a>"

View File

@ -728,7 +728,6 @@ posting:
enable_filtered_replies_view: enable_filtered_replies_view:
default: false default: false
client: true client: true
hidden: true
title_prettify: title_prettify:
default: true default: true
locale_default: locale_default: