Project

General

Profile

提案 #1262

photo_supportの改善

Added by cob odo about 1 year ago. Updated about 1 year ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
photo_support
ブランチ:
topic/1262-extend-photo-support

Description

  • Fotologの画像が展開できなくなっていたので正規表現を修正
  • OpenGraphProtocol(OGP)のog:imageで取得するコードを共通化
    • ついでにOGP取得コードとTwitter Card取得コードを共通化
  • OGP対応サイトとしてマシュマロ https://marshmallow-qa.com/ のdefimageopenerを追加

というようなパッチを書いたので添付します。

追加したメソッド名には改善の余地があります。


Files

photo_support_ogp.patch (3.35 KB) photo_support_ogp.patch cob odo, 2018-06-11 23:43
photo_support_ogp.patch (3.69 KB) photo_support_ogp.patch cob odo, 2018-06-27 17:42

Associated revisions

Revision c79fa4be (diff)
Added by cob odo about 1 year ago

photo_supportプラグインの改善 refs #1262
  • OpenGraphProtocol(OGP)のog:imageで取得するコードを共通化
    • ついでにOGP取得コードとTwitter Card取得コードを共通化
  • Twitter Cardの新しい(?)仕様に追従
  • Fotologの画像が展開できなくなっていたので正規表現を修正
  • OGP対応サイトとしてマシュマロ https://marshmallow-qa.com/ を追加
  • Twitter Card対応サイトとしてpeing https://peing.net/ja を追加

History

#1

Updated by cob odo about 1 year ago

追加でpeingの対応をしようとして気づきましたが、現在のTwitter Cardでは twitter:image:src という属性は存在せず、 twitter:image で取得すべきかもしれません。

https://developer.twitter.com/en/docs/tweets/optimize-with-cards/overview/summary-card-with-large-image

互換性を考え、まず twitter:imageで取得し、見つからなければ twitter:image:srcでフォールバックするように d250g2メソッドを改修したパッチを添付します。(peing対応も含まれています)

複数の目的が混ざったパッチになってしまっているので、分離したほうがよければ分離します。

#2

Updated by toshi_a 初音 about 1 year ago

  • Status changed from 新規 to 実装待ち
  • Assignee set to cob odo

ブランチ作って、commitしてもらっていいですよ。

複数の目的が混ざったパッチになってしまっているので、分離したほうがよければ分離します。

確かに分離している方が丁寧ですが、いいかなという気もします。マシュマロについては知りませんでしたが、俺はTwitterやってないのでなんとも言えませんね……。

追加したメソッド名には改善の余地があります。

めっちゃいいと思いますよ!

#3

Updated by cob odo about 1 year ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee changed from cob odo to toshi_a 初音

topic/1262-extend-photo-supportブランチにpushしました。
担当はとりあえずtoshi_aさんにしておきます。よろしくお願いします。

#4

Updated by cob odo about 1 year ago

  • Tracker changed from 機能 to 提案
  • Start date deleted (2018-06-11)
  • ブランチ set to topic/1262-extend-photo-support
#5

Updated by cob odo about 1 year ago

  • Status changed from レビュー待ち to マージ待ち

一旦はレビューしてもらっているということでマージ待ちへ

#6

Updated by cob odo about 1 year ago

  • Status changed from マージ待ち to 終了

masterへpushしたので終了

Also available in: Atom PDF