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

ブログ記事にサムネイルを設定しても、一覧に表示されない。 #7758

Closed
machida opened this issue May 7, 2024 · 6 comments
Assignees

Comments

@machida
Copy link
Member

machida commented May 7, 2024

貼り付けた画像_2024_05_07_13_56

再現方法

メンターでログインし、 /articles/new にアクセス。

サムネイルは自分でアップロードせず選択する。

貼り付けた画像_2024_05_07_13_57 貼り付けた画像_2024_05_07_13_58

サムネイルを本文に表示しないようにする。

@Shrimprin
Copy link
Contributor

@komagata @machida
お疲れ様です。仕様について2点確認させていただけますでしょうか。

1. 「サムネイル画像を本文に表示」のチェックは今回のバグに影響しますでしょうか?

試したところ、チェックのON, OFFに関わらず、最新記事一覧にはサムネイルが表示されませんでした。
今回のイシューはサムネイル画像を自分でアップロードせずにサンプルから選択した際に、最新記事一覧にサムネイルが表示されないバグを修正するという認識で良いでしょうか?

2. サムネイルの代替メッセージはブログ一覧ページ(/articles])と同様にすればよいでしょうか?

最新記事一覧の代替メッセージを設定するにあたり、ブログ一覧ページの代替メッセージを参考にしようとしたところ、少々違和感があったため確認をさせてください。

現状のブログ一覧ページの代替メッセージ

サムネイルの種類 代替メッセージ
自分でアップロードした画像 ブログ記事「記事のタイトル」のアイキャッチ画像
サンプルから選択した画像 ブログ記事のブランクアイキャッチ画像
何も画像を選択していない ブログ記事「記事のタイトル」のアイキャッチ画像

サンプルから選択した画像の時にブランクアイキャッチ画像となっていたり、何も画像を選択していないときにブランクアイキャッチ画像となっているなど実際に表示されている内容と少し異なると感じました。

実装案

上記を踏まえて実装案を考えましたので、どの実装にすべきがご助言いただけますでしょうか

実装案 1

現状のブログ一覧ページのまま

サムネイルの種類 代替メッセージ
自分でアップロードした画像 ブログ記事「記事のタイトル」のアイキャッチ画像
サンプルから選択した画像 ブログ記事のブランクアイキャッチ画像
何も画像を選択していない ブログ記事「記事のタイトル」のアイキャッチ画像

実装案 2

実際の表示に合わせる

サムネイルの種類 代替メッセージ
自分でアップロードした画像 ブログ記事「記事のタイトル」のアイキャッチ画像
サンプルから選択した画像 ブログ記事のサンプルアイキャッチ画像
何も画像を選択していない ブログ記事のブランクアイキャッチ画像

実装案 3

実際の表示に合わせ、さらにサンプルから選択した画像の場合はArticle.thumbnail_typeに応じたメッセージにする
(例: thumbnail_typerubyならブログ記事のrubyアイキャッチ画像)

サムネイルの種類 代替メッセージ
自分でアップロードした画像 ブログ記事「記事のタイトル」のアイキャッチ画像
サンプルから選択した画像 ブログ記事のthumbnail_typeアイキャッチ画像
何も画像を選択していない ブログ記事のブランクアイキャッチ画像

実装案 4

全て記事のタイトルで統一

サムネイルの種類 代替メッセージ
自分でアップロードした画像 ブログ記事「記事のタイトル」のアイキャッチ画像
サンプルから選択した画像 ブログ記事「記事のタイトル」のアイキャッチ画像
何も画像を選択していない ブログ記事「記事のタイトル」のアイキャッチ画像

自身の考え

私としては以下2点の理由で実装案4が良いと考えています。

  • 見る人としては何のブログのアイキャッチか示された方が直感的にわかりやすい
  • 実装がシンプルになる

以上、お手数ですが誤解お願いいたします 🙏

@machida
Copy link
Member Author

machida commented May 25, 2024

@Shrimprin

1

その認識でOKですー。
サムネイルを選択した場合、自分でアップロードした場合、それが一覧で表示されることを期待しています。それは本文に表示・非表示は関係なく一覧では表示されるようにしてください。

2

4でお願いしますー

@Shrimprin
Copy link
Contributor

@machida
ご回答いただきありがとうございます!
承知いたしました!

@Shrimprin
Copy link
Contributor

@komagata @machida
お疲れ様です!
こちらのイシュー、ポイントが記載されていないのですがポイント1であっていますでしょうか?

お手数ですがご確認よろしくお願いいたします 🙏

@komagata komagata added the 1 label Jun 7, 2024
@komagata
Copy link
Member

komagata commented Jun 7, 2024

@Shrimprin 1ポイントで割り振りました~

@Shrimprin
Copy link
Contributor

@komagata
ありがとうございます!

@komagata komagata moved this to 完成 in bootcamp Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

3 participants