プロジェクト

全般

プロフィール

提案 #1262

完了

photo_supportの改善

cob odo さんがほぼ6年前に追加. 5年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
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

cob odo さんが5年以上前に更新

追加で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対応も含まれています)

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

toshi_a 初音 さんが5年以上前に更新

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

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

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

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

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

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

cob odo さんが5年以上前に更新

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

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

cob odo さんが5年以上前に更新

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

cob odo さんが5年以上前に更新

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

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

cob odo さんが5年以上前に更新

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

masterへpushしたので終了

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