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

未ログイン時のQ&A個別ページのtitleタグの文字数を制限 #7991

Merged
merged 5 commits into from
Aug 21, 2024

Conversation

MikotoMakizuru
Copy link
Contributor

@MikotoMakizuru MikotoMakizuru commented Aug 3, 2024

Issue

概要

  • 未ログイン時のQ&Aの個別ページのtitleタグのタイトルを省略しました。

変更確認方法

  1. chore/truncate-qa-title-in-title-tag-logged-out をローカルに取り込む
  2. ログインして、長いタイトルの質問を作成する
    • 長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイトルの質問
  3. 作成した Q&A 個別ページの URL をコピーしておく
  4. 一度ログアウトして、4 でコピーした URL に遷移
  5. 右クリックで「ページのソースを表示」を選択し、HTML が表示される画面へ
  6. 2 行目の title タグの質問が以下のように省略されていることを確認
    • <title>(development) Q&amp;A: 長いタイトルの質問長いタイトルの質問長いタイトルの質問長いタイト... | FBC</title>
  7. 4 行目 og:title 、7 行目 og:description、12 行目 twitter:description のタイトルはそのまま表示されていることを確認

Screenshot

変更前

スクリーンショット 2024-07-30 0 42 37のコピー3

変更後

スクリーンショット 2024-07-30 0 41 47のコピー2

スクリーンショット 2024-07-30 0 41 47のコピー

@@ -135,3 +135,11 @@ question15:
practice: practice12
created_at: "2022-01-17"
published_at: "2022-01-17"

Copy link
Contributor

Choose a reason for hiding this comment

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

#7990 と同じコードになるため、#7990がマージされたらこの記述は不要になると思います。
このままで問題ありません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

#7990 がマージされたら削除しておきます。

Copy link
Contributor

@kyokucho1989 kyokucho1989 left a comment

Choose a reason for hiding this comment

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

@MikotoMakizuru
確認しました! 問題ありません。Approveしますー

@MikotoMakizuru
Copy link
Contributor Author

@kyokucho1989
レビューありがとうございます!

@MikotoMakizuru MikotoMakizuru force-pushed the chore/truncate-qa-title-in-title-tag-logged-out branch from 3768a5f to bd49ba8 Compare August 20, 2024 17:28
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 02a4bd1 into main Aug 21, 2024
4 checks passed
@komagata komagata deleted the chore/truncate-qa-title-in-title-tag-logged-out branch August 21, 2024 00:27
@github-actions github-actions bot mentioned this pull request Aug 21, 2024
17 tasks
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.

3 participants