Moin
June 10, 2026, 6:44am
1
I think the ads on a topic with nested replies could still be shown in one line, just like they appear in other topics. Right now, they feel much more annoying in topics with nested replies.
While I was looking at the topic again to take a more recent screenshot of the ads for this report, I noticed that sometimes ads seem not to get the space they need. I noticed it on a post starting with a quote and a post with a poll.
8 Likes
Falco
June 10, 2026, 2:39pm
2
Thanks for the reminder, I tried fixing this a few weeks ago, but my fix didn’t quite work.
6 Likes
Falco
June 11, 2026, 2:15pm
3
LMAO, I, like you with the advertising Discourse Advertising Plugin (Ads) , and spent way too much time fussing around there, but turns out this is handled with the GitHub - discourse/discourse-custom-topic-list-ads-component · GitHub instead.
With that out of the way, here are the fixes
main ← fix-nested-replies-view-ads
opened 05:14PM - 10 Jun 26 UTC
Previously, the between-posts ad was rendered through the `post-article` wrapper… outlet, which the nested replies view renders for the OP and for every reply at every depth — so ads appeared inside the OP card and scattered throughout the reply tree, gated by `post_number`, which does not reflect display order in that view.
This change skips the between-posts ad in the nested replies view (via `shouldRender` on the `nestedReplyView` outlet arg) and adds an `AdBetweenNestedRoots` component rendered in the core `nested-roots-between` outlet, which shows an ad after every `show_between_posts` top-level replies instead.
On Discourse versions without the `nested-roots-between` outlet the new component never renders, and the regular topic view is unchanged.
main ← nested-roots-ad-cadence-setting
opened 02:10PM - 11 Jun 26 UTC
Previously, ads between top-level replies in the nested replies view reused `sho… w_between_posts`, but that cadence counts all posts in the classic view while only roots count in the nested view — with `show_between_posts: 15`, a topic with hundreds of posts but few top-level replies showed barely any ads.
This change adds a `show_between_nested_roots` setting (default 5) so the nested view cadence can be tuned independently, with 0 disabling nested-view ads.
Follow-up to #25.
2 Likes
