Project

General

Profile

Actions

提案 #1262

closed

photo_supportの改善

Added by cob odo over 6 years ago. Updated about 6 years ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
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
Actions #1

Updated by cob odo about 6 years 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対応も含まれています)

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

Actions #2

Updated by toshi_a 初音 about 6 years ago

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

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

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

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

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

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

Actions #3

Updated by cob odo about 6 years ago

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

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

Actions #4

Updated by cob odo about 6 years ago

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

Updated by cob odo about 6 years ago

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

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

Actions #6

Updated by cob odo about 6 years ago

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

masterへpushしたので終了

Actions

Also available in: Atom PDF