From 1aa780624499c2ef947481ba4a3432df68d5cdf4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ng=C3=B4=20Qu=E1=BB=91c=20=C4=90=E1=BA=A1t?= Date: Fri, 19 Jan 2024 18:04:25 +0700 Subject: [PATCH] Fix notify for all posts switch field loading state (#3938) --- .../js/src/forum/addSubscriptionSettings.tsx | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/extensions/subscriptions/js/src/forum/addSubscriptionSettings.tsx b/extensions/subscriptions/js/src/forum/addSubscriptionSettings.tsx index 3ad961dd8..192272b56 100644 --- a/extensions/subscriptions/js/src/forum/addSubscriptionSettings.tsx +++ b/extensions/subscriptions/js/src/forum/addSubscriptionSettings.tsx @@ -29,8 +29,14 @@ export default function () { id="flarum_subscriptions__notify_for_all_posts" state={!!this.user!.preferences()?.['flarum-subscriptions.notify_for_all_posts']} onchange={(val: boolean) => { - this.user!.savePreferences({ 'flarum-subscriptions.notify_for_all_posts': val }); + this.notifyForAllPostsLoading = true; + + this.user!.savePreferences({ 'flarum-subscriptions.notify_for_all_posts': val }).then(() => { + this.notifyForAllPostsLoading = false; + m.redraw(); + }); }} + loading={this.notifyForAllPostsLoading} > {app.translator.trans('flarum-subscriptions.forum.settings.notify_for_all_posts_label')}