feat(sticky): make excerpt optional in sticky (#4016)

flarum · Oct 21, 2024 · a0311df · a0311df
1 parent b86a4a0
commit a0311df
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 9 deletions.
diff --git a/extensions/sticky/extend.php b/extensions/sticky/extend.php
@@ -31,6 +31,10 @@
 
     new Extend\Locales(__DIR__.'/locale'),
 
+    (new Extend\Settings())
+        ->serializeToForum('excerptDisplayEnabled', 'flarum-sticky.enable_display_excerpt', 'boolval')
+        ->default('flarum-sticky.enable_display_excerpt', true),
+
     (new Extend\Model(Discussion::class))
         ->cast('is_sticky', 'bool'),
 

diff --git a/extensions/sticky/js/src/admin/extend.tsx b/extensions/sticky/js/src/admin/extend.tsx
@@ -15,11 +15,22 @@ export default [
       'moderate',
       95
     )
-    .setting(() => ({
-      setting: 'flarum-sticky.only_sticky_unread_discussions',
-      name: 'onlyStickyUnreadDiscussions',
-      type: 'boolean',
-      label: app.translator.trans('flarum-sticky.admin.settings.only_sticky_unread_discussions_label'),
-      help: app.translator.trans('flarum-sticky.admin.settings.only_sticky_unread_discussions_help'),
-    })),
+    .setting(
+      () => ({
+        setting: 'flarum-sticky.only_sticky_unread_discussions',
+        name: 'onlyStickyUnreadDiscussions',
+        type: 'boolean',
+        label: app.translator.trans('flarum-sticky.admin.settings.only_sticky_unread_discussions_label'),
+        help: app.translator.trans('flarum-sticky.admin.settings.only_sticky_unread_discussions_help'),
+      }),
+      90
+    )
+    .setting(
+      () => ({
+        type: 'switch',
+        setting: 'flarum-sticky.enable_display_excerpt',
+        label: app.translator.trans('flarum-sticky.admin.settings.enable_display_excerpt'),
+      }),
+      100
+    ),
 ];
diff --git a/extensions/sticky/js/src/forum/addStickyExcerpt.js b/extensions/sticky/js/src/forum/addStickyExcerpt.js
@@ -8,15 +8,15 @@ import { truncate } from 'flarum/common/utils/string';
 
 export default function addStickyExcerpt() {
   extend(DiscussionListState.prototype, 'requestParams', function (params) {
-    if (app.current.matches(IndexPage) || app.current.matches(DiscussionPage)) {
+    if (app.forum.attribute('excerptDisplayEnabled') && (app.current.matches(IndexPage) || app.current.matches(DiscussionPage))) {
       params.include.push('firstPost');
     }
   });
 
   extend(DiscussionListItem.prototype, 'infoItems', function (items) {
     const discussion = this.attrs.discussion;
 
-    if (discussion.isSticky() && !this.attrs.params.q && !discussion.lastReadPostNumber()) {
+    if (app.forum.attribute('excerptDisplayEnabled') && discussion.isSticky() && !this.attrs.params.q && !discussion.lastReadPostNumber()) {
       const firstPost = discussion.firstPost();
 
       if (firstPost) {

diff --git a/extensions/sticky/locale/en.yml b/extensions/sticky/locale/en.yml
@@ -7,6 +7,7 @@ flarum-sticky:
   # Translations in this namespace are used by the admin interface.
   admin:
     settings:
+      enable_display_excerpt: Show an excerpt of the first post when a sticky discussion is unread
       only_sticky_unread_discussions_label: Only sticky unread discussions
       only_sticky_unread_discussions_help: On the All Discussions page, unread sticky discussions pin to the top, while read sticky discussions follow the regular order.