Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ogp.rb: 「最初の画像」を取得するときのスキームなしパスの対処 #216

Open
kmuto opened this issue Dec 31, 2019 · 3 comments

Comments

@kmuto
Copy link
Contributor

kmuto commented Dec 31, 2019

images.first[0]

OGPに記事内の最初のimg srcの値を使うようになっていますが、これが/example.jpgのようにhttp/https等のスキームが付いていないパスの場合はそのまま/example.jpgがOGP値となってしまいます(当然貼り付けると壊れたものになります)。
/から始まらない相対パスのときにどうするのかという問題はありますが、スキームが付いていないときには自ホストURLを付けるといった形にはできるでしょうか。

(あとはOGPを最初以外からも選びたい…という希望ありますが、これはなかなか難しいですよね…)

@tdtds
Copy link
Member

tdtds commented Jan 9, 2020

これはどういうシチュエーションですか? imageプラグイン等を使わずに直接img要素を書いている感じです? (要件としてはよくわかります。対応も可能だとは思います)

(あとはOGPを最初以外からも選びたい…という希望ありますが、これはなかなか難しいですよね…)

そうですね……編集中にJavaScriptで画像を列挙して選ばせるようなUIは考えつきますが、プレビューの存在を前提にしなくてはならないなど、クリアしなければならない条件がけっこうたいへんです。そもそも1ページに複数のエントリがある日記形式とogimageは相性が悪いので、あまりリッチな対応は労力に見合わない感がありまして……。

@tdtds tdtds added this to To do in 5.1.1リリース via automation Jan 9, 2020
@kmuto
Copy link
Contributor Author

kmuto commented Jan 9, 2020

そうか、imageプラグインを使えばよかったですね。記事のほうではWikiスタイルで

http:/dtpadv2019/sample.png

みたいにして、「画像の場合はそのまま使われる」の機能に委ねていたのでした。

@tdtds
Copy link
Member

tdtds commented Jan 9, 2020

うーん、その記法にちゃんと対応するのはけっこうたいへんそうですねぇ🤔

@tdtds tdtds removed this from To do in 5.1.1リリース Feb 29, 2020
@tdtds tdtds added this to To do in 5.1.2リリース via automation Feb 29, 2020
@tdtds tdtds removed this from To do in 5.1.2リリース May 31, 2020
@tdtds tdtds added this to To do in 5.1.3リリース via automation May 31, 2020
@tdtds tdtds added this to To do in 5.1.4リリース via automation Aug 29, 2020
@tdtds tdtds removed this from To do in 5.1.3リリース Aug 29, 2020
@tdtds tdtds removed this from To do in 5.1.4リリース Nov 29, 2020
@tdtds tdtds added this to To do in 5.1.5リリース via automation Nov 29, 2020
@tdtds tdtds removed this from To do in 5.1.5リリース Feb 28, 2021
@tdtds tdtds added this to To do in 5.1.6リリース via automation Feb 28, 2021
@tdtds tdtds removed this from To do in 5.1.6リリース May 29, 2021
@tdtds tdtds added this to To do in 5.1.7リリース via automation May 29, 2021
@tdtds tdtds removed this from To do in 5.1.7リリース Aug 29, 2021
@tdtds tdtds added this to To do in 5.2.0リリース via automation Aug 29, 2021
@tdtds tdtds removed this from To do in 5.2.0リリース Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants