Onebox Twitter embedding

Oh, thanks! Is it possible to do that without actually allowing Twitter logins?

Disable enable twitter logins and just provide twitter consumer key & twitter consumer secret.

3 curtidas

Tried it. It didn’t work. Embedded tweets still do not contain images.

Some long tweets with images don’t work

Shorter tweets with images works

It exists since the new twitter policy regarding links and images in the 140 characters count

6 curtidas

I’ve just submitted a PR which should fix this:

10 curtidas

@david - thank you! We do a lot of Twitter oneboxing and found the truncation very annoying.

4 curtidas

This is terrific - my forum users really need this.

Is this merged into a future release or something we have to do to modify in our existing installations?

Just upgrade on latest version, it should be good :

  • via yourforum.com/admin/upgrade
  • or SSH into your server and do these commands :

cd /var/discourse
git pull
./launcher rebuild app

I don’t think you need to add the twitter consumer key and twitter consumer secret in your admin settings but I’m only 99% sure

2 curtidas

Yep - it’s all merged into master and working great on my forum

I’m afraid you will need to add these keys to your site settings - the data is fetched via the twitter API rather than opengraph/oembed.

2 curtidas

It’s always the 1% left

1 curtida

I just pushed a small improvement for Twitter onebox. The onebox will now show like and retweet count.

11 curtidas

Twitter onebox with videos are really long on mobile. Here’s one I am adding to this post.

How it appears on mobile:

5 curtidas

@awesomerobot is there a simple CSS fix for :arrow_double_up:

4 curtidas

At the moment on mobile we set a max-height (70vh) and 100% width, and the video scales to fit. This works better for portrait oriented videos than landscape.

We can assume most videos will be landscape, but then we crop portrait oriented videos pretty severely as a trade-off.

There’s no simple fix for both without knowing the aspect ratio (Are we able to calculate the aspect ratio of these and maybe append a style based on that? the iframe seems to come with height/width).

4 curtidas

Yes, we can do that. This is the line where we build the iframe

What HTML structure would be most useful for you @awesomerobot? Something like

<div class='tweet-images'>
  <div class='aspect-image-full-size' style='--aspect-ratio:#{width}/#{height};'/>
    <iframe class='tweet-video' src='https://twitter.com/i/videos/blah' width='123' height='456' frameborder='0' allowfullscreen></iframe>
  </div>
</div>

And then we can re-use the aspect-image-full-size logic which we use for images?

1 curtida

Yeah I think that could work, with the aspect ratio there I should be able to adjust as needed.

2 curtidas

Estou tentando descobrir como fazer com que tweets puxados do Zapier apareçam como um tópico assim, mas estou com dificuldades. Alguém pode me indicar o caminho? Funcionaria se eu apenas ativar o ‘onebox’ nas configurações?

3 curtidas

Existe alguma maneira de fazer com que tweets incorporados mostrem o tweet ao qual estão respondendo? Por exemplo, se eu incorporar este tweet https://twitter.com/AOC/status/1230329539924025345, no phpBB e no vBulletin, ele exibe tanto o tweet quanto o conteúdo ao qual ela está respondendo.
phpBB:

vB:

Mas no Discourse, apenas a resposta é mostrada, o que muitas vezes não faz sentido sem o contexto do tweet original

Claro, é fácil o suficiente para postar o tweet original junto com a resposta, mas a maioria dos usuários (e alguns bots do Twitter que usamos) não são espertos/conscientes o suficiente para perceber que precisam fazer isso.

7 curtidas

Este é um problema muito complicado: o código de incorporação “oficial” do Twitter apresenta dois problemas:

  1. Ele não fornece a altura, o que faz com que os tópicos se movam enquanto carregam.
  2. Seria necessário executar os arquivos JavaScript do Twitter em todos os nossos sites.

Contornamos isso usando nosso próprio renderizador personalizado, mas, infelizmente, isso significa que precisamos constantemente nos atualizar em relação ao código de incorporação oficial.

6 curtidas