ブラウザのタブのタイトルとページの種類が一致しないのを修正

hagiya0121 commented 1 month ago

Issue

未ログインの状態で特定のページにアクセスすると、ブラウザのタブのタイトルがページの種類と一致しない問題を修正しました。

_unauthorized_header.html.slimというパーシャルで間違ったタイトルが設定されていました。

- title "Q&A: #{truncate(title, length: 35, omission: '...')}"
- set_meta_tags og: { title: "Q&A: #{title}" }

原因のパーシャルを呼び出しているunauthorized_show.slimでもタイトルを設定しているので、パーシャル側のタイトル設定は不要だと思い削除しました。
set_meta_tags og: { title: "Q&A: #{title}" }は無くても、デフォルトでog:titleにtitleタグの内容が入るように設定されていたので削除しました。
unauthorized_show.slimのタイトル設定をログイン時のタイトルと一致するように修正しました。

bug/fix-tab-titleをローカルに取り込む
foreman start -f Procfile.devでサーバーを立ち上げ
未ログイン状態でプラクティスの詳細ページにアクセスhttp://localhost:3000/practices/315059988
タブのタイトルが、ログイン状態のタブのタイトル：(development) プラクティス OS X Mountain Lionをクリーンインストールする | FBCと一致することを確認
未ログイン状態でDocsの詳細ページにアクセスhttp://localhost:3000/pages/1069065928
タブのタイトルが、ログイン状態のタブのタイトル：(development) Docs: プラクティスに紐付いたDocs3 | FBCと一致することを確認
未ログイン状態でQ&Aの詳細ページにアクセス http://localhost:3000/questions/1037106414
タブのタイトルが、ログイン状態のタブのタイトル：(development) Q&A: テストの質問40 | FBCと一致することを確認
未ログイン状態でユーザーの詳細ページにアクセス http://localhost:3000/users/991528156
タブのタイトルが、ログイン状態のタブのタイトル：(development) kimuraさんのプロフィール | FBCと一致することを確認

hagiya0121 commented 1 month ago

@Ryooo-k お疲れ様です。こちらのPRのレビューをお願いしたいです🙏 ご都合が悪いときは遠慮なくおっしゃってください🙇

Ryooo-k commented 1 month ago

@hagiya0121 承知しました！レビューさせていただきます👍 今週中を目処にご連絡いたします！

Ryooo-k commented 1 month ago

@hagiya0121 確認しました！動作としては問題ないと思います👍 何点かコメントいたしましたので、ご確認お願いしまーす！

hagiya0121 commented 1 month ago

@Ryooo-k レビューありがとうございます🙏 指摘いただいた箇所を修正したので確認お願いします。

hagiya0121 commented 4 weeks ago

@Ryooo-k レビューありがとうございます🙇

@komagata メンバーのレビューが完了したのでレビューお願いします。