Moin
Juin 10, 2026, 6:44
1
Je pense que les publicités dans un sujet avec des réponses imbriquées pourraient toujours s’afficher sur une seule ligne, tout comme elles apparaissent dans d’autres sujets. Pour l’instant, elles semblent beaucoup plus gênantes dans les sujets avec des réponses imbriquées.
Alors que j’examinais à nouveau le sujet pour prendre une capture d’écran plus récente des publicités pour ce rapport, j’ai remarqué que parfois les publicités ne semblent pas obtenir l’espace dont elles ont besoin. Je l’ai remarqué dans un message commençant par une citation et dans un message avec un sondage.
8 « J'aime »
Falco
(Falco)
Juin 10, 2026, 2:39
2
Merci pour le rappel. J’ai essayé de régler ce problème il y a quelques semaines, mais ma solution n’a pas tout à fait fonctionné.
6 « J'aime »
Falco
(Falco)
Juin 11, 2026, 2:15
3
LMAO, moi aussi, comme toi avec le tag advertising , je pensais que ça était géré par le Plugin Discourse Advertising (Ads) , et j’ai passé bien trop de temps à trifouiller là-dedans, mais il s’avère que c’est géré par le GitHub - discourse/discourse-custom-topic-list-ads-component · GitHub à la place.
Une fois cela réglé, voici les correctifs
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 « J'aime »