Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

カード内の見出しがh3から開始している #6404

Closed
yosukedoke opened this issue Jun 8, 2021 · 5 comments · Fixed by #6528
Closed

カード内の見出しがh3から開始している #6404

yosukedoke opened this issue Jun 8, 2021 · 5 comments · Fixed by #6528
Labels
accessibility アクセシビリティ関連 bug バグ。本来あるべき動作をしていないもの help-wanted 特に助けを必要としているもの

Comments

@yosukedoke
Copy link

起こっている問題 / The Problem

  • カード系の個別ページの見出しがh1(サイト名)の次がh3でレベルがスキップしている

期待する見せ方・挙動 / Expected Behavior

  • 1.3.1的には、レベルはh2から始まるほうが望ましいと思われる
    • トップページでレンダリングされる場合と個別URLでレンダリングされるケースにおいて同一コンポーネントを利用してることでレベルの段階が変わっていると思われます。
      • 許容しても大きな問題ではないかもしれないが、(実装的には)同一のコンポーネントで実装されているがゆえに文脈によりコンポーネントの要素を h2/h3 で切り替える等で改善は可能と思われるのでご検討ください

起こっている問題の再現手段 / Steps to Reproduce

  1. ブラウザのデベロッパーツール等で該当部分のソースを確認する

動作環境・ブラウザ / Environment

  • すべての環境
@yosukedoke yosukedoke added the bug バグ。本来あるべき動作をしていないもの label Jun 8, 2021
@masuP9 masuP9 added the accessibility アクセシビリティ関連 label Jun 8, 2021
@kaizumaki
Copy link
Collaborator

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

文脈によりコンポーネントの要素を h2/h3 で切り替える

この方向で対応するのがよさそうですね。

@kaizumaki kaizumaki added the help-wanted 特に助けを必要としているもの label Jun 8, 2021
@bakera
Copy link

bakera commented Jun 18, 2021

このようなケース、発想の転換で、上位のh2見出しを追加することで解決できる場合もあります。
カード型コンポーネントが並ぶシチュエーションでは、それらを括る見出しがあった方が理解しやすくなるケースが多いです。
ご参考までに……。

@kaizumaki
Copy link
Collaborator

こちら #6415 との兼ね合いを考えると、 @bakera さんのご提案のように上位のh2見出しを追加する方向がいいような気がしてきました。

@kaizumaki
Copy link
Collaborator

ドラフトで #6528 を立ててみました。 @bakera さんのコメントのようにh2タイトルを追加する作戦です。
セマンティック的にいけてるのかちょっとわからないのですが、これはどうなんでしょう? @magi1125 @masuP9
(以下プレビュー)
https://deploy-preview-6528--dev-covid19-tokyo.netlify.app/cards/number-of-confirmed-cases

もしこれでいけるということであれば、コンテンツの修正になるので東京都に確認をとります。

@magi1125
Copy link

@kaizumaki おお、よさそうです!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility アクセシビリティ関連 bug バグ。本来あるべき動作をしていないもの help-wanted 特に助けを必要としているもの
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants