Twitter quote-tweets not shown in oneboxes

Twitter oneboxes work well, except for quote tweets. In this case, the “parent” tweet is shown, but the quoted tweet is only shown as a link.

Would appreciate if someone could improve the oneboxing of quote tweets.

Quote tweet as displayed on Twitter:

Onebox:

11 лайков

I like this suggestion, its a rather complex change, but wacking a pr-welcome on it for now in case anyone is feeling brave.

5 лайков

I’ll try to make a PR to fix this. Will let you know

9 лайков

Still working on it but let me know if you have any suggestions/comments :slight_smile:

13 лайков

PR уже почти готов:

@techAPJ проводит проверку.

Есть несколько мелких моментов по CSS, которые @awesomerobot, возможно, захочет посмотреть. В целом мы не очень любим использовать :hover, а скруглённые углы там выглядят больше, чем у Twitter, но это легко исправить.

В целом всё выглядит отлично, @jcalvento!

7 лайков

PR выглядит отлично, @jcalvento! :+1:

Оставил небольшой комментарий по поводу форматирования кода. Как только это будет исправлено, можно сливать. :slight_smile:

4 лайка

Это изменение уже в работе :tada:

Спасибо @jcalvento :trophy:

10 лайков

На мобильных устройствах отступы кажутся чрезмерными. Можно ли их уменьшить?

6 лайков

Я внес некоторые изменения, чтобы улучшить общее оформление цитат.

На мой взгляд, всё ещё есть небольшой резерв для уменьшения отступов между:

  • именем/именем пользователя и текстом твита
  • цитируемым твитом и временем публикации/лайками/ретвитами

Эти отступы возникают из-за свойства white-space: pre-line, применённого к классу tweet. @awesomerobot, не могли бы вы быстро взглянуть, есть ли возможность для дальнейшего улучшения?

11 лайков

На мобильных устройствах всё ещё слишком много пустого пространства.

1 лайк

Да, это необходимо, чтобы пробелы в твите отображались так, как они были опубликованы… но из-за этого также появляются пробелы в HTML, поэтому здесь я удалил их в шаблоне — это должно исправить проблему.

5 лайков

Обновили gem, и onebox теперь выглядит гораздо лучше!

Стоит ли уменьшить отступ сверху у временной метки до 5px (вместо 10px)?

8 лайков

О, кажется, я пропустил несколько пробелов внизу… Думаю, это исправит ситуацию.

5 лайков

Не знаю, думаю, Арпит прав, когда говорит

2 лайка

Здесь всё ещё слишком много пробелов @awesomerobot

3 лайка

Хм, в шаблоне где-то остался лишний перенос строки, который я упустил… но удаление отступов всё равно уберёт пробел, несмотря на это.

5 лайков