Moin
10.Июнь.2026 06:44:58
1
Я думаю, что реклама в теме с вложенными ответами всё ещё может отображаться в одну строку, как и в других темах. Сейчас она кажется гораздо более раздражающей в темах с вложенными ответами.
Пока я снова просматривал тему, чтобы сделать более свежий скриншот рекламы для этого отчёта, я заметил, что иногда реклама не получает необходимого пространства. Я обратил на это внимание в сообщении, начинающемся с цитаты, и в сообщении с опросом.
8 лайков
Falco
(Falco)
10.Июнь.2026 14:39:22
2
Спасибо за напоминание. Я пытался исправить это несколько недель назад, но моё исправление не совсем сработало.
6 лайков
Falco
(Falco)
11.Июнь.2026 14:15:05
3
Ахаха, я, как и ты с тегом advertising , думал, что это обрабатывается через Discourse Advertising Plugin (Ads) , и потратил слишком много времени на возню там, но оказалось, что это обрабатывается с помощью GitHub - discourse/discourse-custom-topic-list-ads-component · GitHub .
С этим разобрались, вот исправления
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 лайка