Topic embedding needs some love

[sam (Sam Saffron)]

@simon

• Semantic Related Topics title missing
• Community Sentiment title missing
• missing unordered list in the Modules Providers section
• Installing Discourse AI on your community title missing

I think this is all happening even in readablity.js, this is firefox reader view:

<h2 id="installing-discourse-ai-on-your-community">
      <strong>Installing Discourse AI on your community</strong>
</h2>

@GPT_Bot

The HTML you provided is technically correct, but it is not a best practice to use <strong> inside an <h2> tag. The <h2> tag already implies a higher level of importance and is typically styled to be bold by default. Using <strong> inside the heading is redundant and can lead to unnecessary complexity in your code.

A better approach would be:

<h2 id="installing-discourse-ai-on-your-community">
    Installing Discourse AI on your community
</h2>

If you want to add specific styling to the <h2> tag, it’s better to use CSS for that purpose.

Will see if there is an easy way to fix this…

@simon

Ideally, the “Sign up for our newsletter” prompt would be excluded from the embedded post.

Not sure about this… but if we really really want to do that we can add .discourse-newsletter-signup to blocked_embed_selectors