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

Change breadcrumb markup from paragraph to ordered list #86

Merged
merged 1 commit into from
Dec 5, 2019

Conversation

r7kamura
Copy link
Contributor

@r7kamura r7kamura commented Dec 5, 2019

HTMLページ上部のパンくずリスト部分のマークアップ方法を、これまでp要素と>の文字で1つの段落として表現していたのを、ol要素とli要素を使って順序付きリストで表現するように変えたいのですが、どうでしょうか。

ナビゲーション用の要素であることが機械からもより理解しやすくなるので、検索インデックス生成用のクローラにも優しいし、将来もう少しメタデータを追記することで、例えばGoogleの検索結果にパンくずリストが表示されるようにもできます。
https://developers.google.com/search/docs/data-types/breadcrumb?hl=ja

@r7kamura
Copy link
Contributor Author

r7kamura commented Dec 5, 2019

見た目は変更前後でほぼ変わらないように気を付けてCSSを編集しています。

良い感じに確認する方法が分からなかったのでかなり雑ですが (もし良い方法を知っていたら教えてください!)、次のような見た目になります。

変更前

image

変更後

image

Copy link
Member

@hanachin hanachin left a comment

Choose a reason for hiding this comment

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

header/navでマークアップするとアクセシビリティの観点でもよさそう 👍

確認、私はbitclustのこのブランチチェックアウトしてdoctreeの方で bundle exec rake して /tmp/html にある生成されたファイル見てます。

見た目、実際見てみて違い感じなかったので、マージしちゃいます。
Pull Request圧倒的感謝 🙏

@hanachin hanachin merged commit 6119e29 into rurema:master Dec 5, 2019
@r7kamura
Copy link
Contributor Author

r7kamura commented Dec 5, 2019

確認、私はbitclustのこのブランチチェックアウトしてdoctreeの方で bundle exec rake して /tmp/html にある生成されたファイル見てます。

おー、rakeのタスクが用意されてるのか。ありがとうございます 👍

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