Skip to content

Commit 6846f42

Browse files
authored
fix: js error as guest on DiscussionListItem (#4056)
1 parent 88f182c commit 6846f42

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

framework/core/js/src/forum/components/DiscussionListItem.tsx

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ export default class DiscussionListItem<CustomAttrs extends IDiscussionListItemA
7474
const discussion = this.attrs.discussion;
7575
const controls = DiscussionControls.controls(discussion, this).toArray();
7676

77-
items.add('controls', this.controlsView(controls), 100);
77+
!!controls.length && items.add('controls', this.controlsView(controls), 100);
7878
items.add('slidableUnderneath', this.slidableUnderneathView(), 90);
7979
items.add('content', this.contentView(), 80);
8080

@@ -83,16 +83,14 @@ export default class DiscussionListItem<CustomAttrs extends IDiscussionListItemA
8383

8484
controlsView(controls: Mithril.ChildArray): Mithril.Children {
8585
return (
86-
!!controls.length && (
87-
<Dropdown
88-
icon="fas fa-ellipsis-v"
89-
className="DiscussionListItem-controls"
90-
buttonClassName="Button Button--icon Button--flat Slidable-underneath Slidable-underneath--right"
91-
accessibleToggleLabel={app.translator.trans('core.forum.discussion_controls.toggle_dropdown_accessible_label')}
92-
>
93-
{controls}
94-
</Dropdown>
95-
)
86+
<Dropdown
87+
icon="fas fa-ellipsis-v"
88+
className="DiscussionListItem-controls"
89+
buttonClassName="Button Button--icon Button--flat Slidable-underneath Slidable-underneath--right"
90+
accessibleToggleLabel={app.translator.trans('core.forum.discussion_controls.toggle_dropdown_accessible_label')}
91+
>
92+
{controls}
93+
</Dropdown>
9694
);
9795
}
9896

0 commit comments

Comments
 (0)