It would be good to use a theme function so that the output of the Tweet embeds could be overridden in the theme layer.

CommentFileSizeAuthor
#2 twitter_api_block-3125828-1.patch1.54 KBdrupalninja99

Issue fork twitter_api_block-3125828

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

drupalninja99 created an issue. See original summary.

drupalninja99’s picture

drupalninja99 commented
Status: Active » Needs review
StatusFileSize
new twitter_api_block-3125828-1.patch1.54 KB

Added a patch that will create the theme function and template.

matthieuscarset made their first commit to this issue’s fork.

matthieuscarset opened merge request !1

littledynamo’s picture

littledynamo commented
Status: Needs review » Reviewed & tested by the community

Just tested this and it works great :)

phily’s picture

phily
he/him
Primary language French
Location 🇪🇺🇫🇷 Paris, France
commented

Patch #2 works for me using Drupal 9.3.3 and Twitter API Block 3.0.0
Thanks

  • matthieuscarset committed cba29a7 on 8.x-2.x 
    Issue #3125828: Add a theme function, template for Tweets markup
matthieuscarset’s picture

matthieuscarset commented
Status: Reviewed & tested by the community » Fixed

Merged! please, forgive me for the delay in answering.

Thank you very much for this contribution.

matthieuscarset’s picture

matthieuscarset commented
Status: Fixed » Closed (fixed)