ブラウザのタブのタイトルとページの種類が一致しないのを修正 #8156
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hagiya0121
force-pushed
the
bug/fix-tab-title
branch
from
7662c89
to
bf3e726
Compare
|
@Ryooo-k |
|
@hagiya0121 |
Ryooo-k
reviewed
Oct 28, 2024
| @@ -8,5 +6,5 @@ header.unauthorized-header | |||
| = label | |||
| h1.unauthorized-header__title | |||
| span.font-thin 「 | |||
| = title | |||
| = truncate(title, length: 35, omission: '...') | |||
There was a problem hiding this comment.
変更前は文字数制限なくtitleをそのまま出力してますが、変更後は文字数制限を設けてます。
この仕様変更は確認済み、という理解でよいでしょうか？
（Q&Aのtitleもレンダリングされるので、変更されたのだと思いますが念の為確認です）
There was a problem hiding this comment.
自分が仕様を勘違いしていました。修正しました！
| title "Q&A: #{truncate(@question.title, length: 35, omission: '...')}" | ||
| set_meta_tags og: { title: "Q&A: #{@question.title}" } | ||
| description "オンラインプログラミングスクール「フィヨルドブートキャンプ」のQ&A「#{@question.title}」のページです。" | ||
| content_for :extra_body_classes |
There was a problem hiding this comment.
ものすごーく細かいことですが、インデントが半角スペース１つだけなのが気になりました！
他のファイルも確認してみましたが、インデントは半角スペース２つで統一されているので、合わせた方が良いかなと思いました👀
|
@hagiya0121 |
|
@Ryooo-k |
Ryooo-k
approved these changes
Oct 30, 2024
9 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue
プラクティスなのにタイトルが"Q&A"になっている #8053
概要
未ログインの状態で特定のページにアクセスすると、ブラウザのタブのタイトルがページの種類と一致しない問題を修正しました。
原因
_unauthorized_header.html.slimというパーシャルで間違ったタイトルが設定されていました。対処
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でサーバーを立ち上げ(development) プラクティス OS X Mountain Lionをクリーンインストールする | FBCと一致することを確認(development) Docs: プラクティスに紐付いたDocs3 | FBCと一致することを確認http://localhost:3000/questions/1037106414
(development) Q&A: テストの質問40 | FBCと一致することを確認http://localhost:3000/users/991528156
(development) kimuraさんのプロフィール | FBCと一致することを確認Screenshot
変更前
変更後