[アクセシビリティ対応] トップページ以外のサイトタイトルは段落にしました

sysbird commented 10 months ago

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

[アクセシビリティ対応] 見出し #208

複数の <h1> は推奨されないため、ヘッダー内のサイトタイトルが全ページで <h1>となっているのを、下記に変更しました【現状】

すべてのページで見出しが <h1> リンクあり

【対応後】

トップページでは見出しでリンクなし
トップページ以外では段落でリンクあり

このプルリクで変更した事を記載してください

logo-title--sns-contact__nav 内で登録しているヘッダーのパターンを2つにわけた
- (1)トップページ以外
- (2)トップページ
(1)のスタイルを追加
トップページ (2)として表示されるページは下記
- ホームページ (home.txt)
- フロントページ (front.html)
- デフォルト (index.html)

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

ソースコードについて

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。書いていない場合は書かない理由を記載してください。

[ ] 書けそうなテストは書いたか？

その他

[x] readme.txt に変更内容は書いたか？
[x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[ ] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

変更内容について何を確認したか、どういう方法で確認をしたかなど

サイトタイトルが、

トップページでは見出しでリンクなしで表示される
トップページ以外では段落でリンクありで表示される

※ トップページとは、ホーム、フロント、デフォルトのテンプレートをさします

レビュワーの確認方法・確認する内容など

上記と同じです

レビュワーに回す前の確認事項

[x] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

確認して変更が反映されていない場合の確認事項

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

sysbird commented 10 months ago

補足

同様の対応をしているテーマ W3CSSPress を参考にしています
ブロックテーマでこの対応しているテーマが上記のみしか見つからなかった
header.html を2つにわけたのが微妙です
ヘッダーパターンのなかで logo-title--sns-contact__nav のみ対応したのは、デフォルトの状態でアクセシビリティ対応していれば審査上はOKだからです
tt 系テーマは非対応(サイトタイトルは常に段落になっている)
tt 系のように、すべてのサイトタイトルを段落で表示し、トップページの「Let's embark on a new journey」を <h1>にする手もあるようです
- 公式ディレクトリでアクセシビリティ対応しているブロックテーマ(12個)を見ると、現状のx-t9と同じのが多い

kurudrive commented 9 months ago

@sysbird ありがとうございます。

ヘッダーがサイトタイトルのためだけに分かれるのは一般ユーザーには理解不能だと思うので避けたいです。

サイトタイトルブロックのデフォルトタグを段落にするだけして、トップページだけ WP_HTML_Tag_Processor で h1 に出来ないか検討しましたが無理でした。

トップページで使用しているヒーローエリアのパターンの元を h1 にすると、パターンを貼り付けられた時に h1 がついてしまうので、それもできないフロントページのヒーローエリアは埋め込みにすると、同じようなものの2元管理になってしまうので望ましくない。

そもそも本質的には Dynamic If Block で対応できる。

公式通すためだけに構造がカオスになるよりはアクセスビリティ対応見送るくらいで良い。

と、いう事でここは対応見送ります。

kurudrive commented 9 months ago

https://github.com/vektor-inc/x-t9/pull/226

vektor-inc / x-t9