プロジェクト

全般

プロフィール

操作
リンクをコピー

提案 #1262

完了

photo_supportの改善

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

ステータス:
終了
優先度:
通常
担当者:
toshi_a 初音
対象バージョン:
3.7
プラグイン名:
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
操作
リンクをコピー
 #1

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

追加で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 初音 さんが6年以上前に更新

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

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

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

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

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

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

操作
リンクをコピー
 #3

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

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

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

操作
リンクをコピー
 #4

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

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

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

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

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

操作
リンクをコピー
 #6

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

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

masterへpushしたので終了

操作
リンクをコピー

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