トップにもどるボタンのリンク先を追加

sysbird commented 3 months ago

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

[アクセシビリティ] トップにもどるボタンのリンク先がない問題 #1094

どういう変更をしたか？

トップにもどるボタンのリンク先は #top がページ内に存在しないのは HTML5の仕様として正しいが、アクセシビリティ的にグレーなため追加しました

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

ソースコードについて

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

デザイン・UI

[x] 初見のユーザーが予備知識無しで使っても使いやすいようになっているか？
[x] 情報意味を考慮した意味グルーピング・余白になっているか？
[x] アラートの表示など追加した場合は他の同様の表示と同じデザインになっているか？

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？
[ ] 表示要素が仕様通りに表示されない不具合の修正ではない or 表示要素に関する不具合修正の場合テストは書いたか？

その他

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

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

トップにもどるボタンが有効の場合、ページ読み込み時にbody要素にID topを追加
すでに body要素にID topがある場合は追加されない
もし body要素にtop以外のIDがある場合は、それを利用する → ボタンのリンク先を変更 → もし IDが複数あったら(ないんだけど)先頭を採用
body要素以外に topが使われているかどうかねんのため確認して、top-1 、top-2…のように重複しないIDを付与 → ボタンのリンク先も変更
Lightning のなかで要素にIDを付けてみたりしながら確認しました

確認URL

（　どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL　）

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

上記に同じです

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

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

レビュワー向け

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

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

drill-lancer commented 3 months ago

@sysbird テスト通りました。

drill-lancer commented 2 months ago

@sysbird npm run build を修正しても反映されなかったので gulp のバージョンを下げたら反映されました。

２人目確認お願いします。

sysbird commented 2 months ago

@kurudrive 確認ありがとうございますそうですね、となると #top がどこかで使われてないか？も確認必要そう、ややこしい〜検討します！

sysbird commented 1 month ago

@kurudrive 修正しました！お手数ですが、ご確認よろしくお願いします

vektor-inc / vk-all-in-one-expansion-unit