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

画像と著者のアイコンに使用できる画像フォーマットを増やす #57

Merged
merged 5 commits into from
Dec 8, 2023

Conversation

ciffelia
Copy link
Member

@ciffelia ciffelia commented Dec 7, 2023

What

  • ブログ著者とタグのアイコン指定方法を変更した。
    • src/content/{authors,tags}に画像ファイルを置き、JSONでファイル名を指定することでアイコンを変更できる。
    • 従来のsrc/assets/icons/blogのファイル名指定と外部URL指定は廃止した。

Why

  • アイコン指定を簡単にするため。
  • 使用できる画像フォーマットを増やすため。
  • 外部URL指定(画像の直リンク)は用途が限られるため。
  • <Icon>コンポーネントの仕様変更にあたって使用箇所を減らすため。

@ciffelia ciffelia changed the title WIP 画像と著者のアイコンに使用できる画像フォーマットを増やす Dec 7, 2023
@ciffelia ciffelia marked this pull request as ready for review December 7, 2023 13:02
Copy link
Contributor

@miyaji255 miyaji255 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

何故かOGPが表示されないです。おそらく拡張子が必要なのだと思います。

また、特にタグのアイコンは2~3色のものが多くjpegよりpngのほうがいいと考えたのですが、やはりjpegのほうが良いですか?

@ciffelia
Copy link
Member Author

ciffelia commented Dec 7, 2023

何故かOGPが表示されないです。おそらく拡張子が必要なのだと思います。

こちらでは拡張子がついており表示されるように見えるのですが、どのページでしょうか?

また、特にタグのアイコンは2~3色のものが多くjpegよりpngのほうがいいと考えたのですが、やはりjpegのほうが良いですか?

たしかにタグのアイコンはPNGで問題なさそうなのでそうします。

@miyaji255
Copy link
Contributor

ogpのパスを直接指定してブラウザで見る分には問題ないですが、下に示すような拡張機能を用いてogpを表示するテストをしてみると少なくともdiscordでは表示されません。また、ogpのパスにビルド時のパスがそのまま見えているのも良くないと思います。

https://chromewebstore.google.com/detail/localhost-ogp-%E3%83%81%E3%82%A7%E3%83%83%E3%82%AB%E3%83%BC/gcbnmkhkglonipggglncobhklaegphgn?hl=ja

@miyaji255
Copy link
Contributor

何故かOGPが表示されないです。おそらく拡張子が必要なのだと思います。

すみません。これはastro devだけでした。

@ciffelia ciffelia merged commit 701fa37 into main Dec 8, 2023
5 checks passed
@ciffelia ciffelia deleted the ciffelia/feat-author-image branch December 8, 2023 14:40
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

Successfully merging this pull request may close these issues.

2 participants