From f92ed88c52645f0ddfddb9b88a7e08b2dc4ceff2 Mon Sep 17 00:00:00 2001 From: David Taylor Date: Thu, 6 Jul 2023 17:02:59 +0100 Subject: [PATCH] FEATURE: Enable loading slider by default (#22439) Followup to d51baa3bb3b08c5a7df6c22593d0162be6543147 Also includes: Force full rerender of post-stream widget when switching topics. This ensures that plugin/theme decorators are re-run when we switch between topics with the loading slider enabled. --- .../discourse/app/components/scrolling-post-stream.js | 2 +- config/site_settings.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/discourse/app/components/scrolling-post-stream.js b/app/assets/javascripts/discourse/app/components/scrolling-post-stream.js index a869581c590..856b26359b8 100644 --- a/app/assets/javascripts/discourse/app/components/scrolling-post-stream.js +++ b/app/assets/javascripts/discourse/app/components/scrolling-post-stream.js @@ -372,7 +372,7 @@ export default MountWidget.extend({ }, didUpdateAttrs() { - this._refresh(); + this._refresh({ force: true }); }, _handleWidgetButtonHoverState(event) { diff --git a/config/site_settings.yml b/config/site_settings.yml index c990fdbd926..ff4a1ebb0ca 100644 --- a/config/site_settings.yml +++ b/config/site_settings.yml @@ -396,7 +396,7 @@ basic: page_loading_indicator: client: true type: enum - default: "spinner" + default: "slider" choices: - spinner - slider