プロジェクト

全般

プロフィール

提案 #1262

photo_supportの改善

cob odo2ヶ月前に追加. 29日前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
photo_support
ブランチ:
topic/1262-extend-photo-support

説明

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

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

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

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

関係しているリビジョン

リビジョン c79fa4be (差分)
cob odo約2ヶ月前に追加

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 を追加

履歴

#1 cob odo約2ヶ月前に更新

追加で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 toshi_a 初音約2ヶ月前に更新

  • ステータス新規 から 実装待ち に変更
  • 担当者cob odo にセット

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

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

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

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

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

#3 cob odo約2ヶ月前に更新

  • ステータス実装待ち から レビュー待ち に変更
  • 担当者cob odo から toshi_a 初音 に変更

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

#4 cob odo30日前に更新

  • トラッカー機能 から 提案 に変更
  • 開始日 を削除 (2018-06-11)
  • ブランチtopic/1262-extend-photo-support にセット

#5 cob odo29日前に更新

  • ステータスレビュー待ち から マージ待ち に変更

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

#6 cob odo29日前に更新

  • ステータスマージ待ち から 終了 に変更

masterへpushしたので終了

他の形式にエクスポート: Atom PDF