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

リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) #2362

Merged
merged 10 commits into from
Dec 6, 2024

Conversation

mtdkei
Copy link
Contributor

@mtdkei mtdkei commented Dec 3, 2024

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

#2199
#2134

どういう変更をしたか?

リンクツールバーにEdit linkを追加し、aタグのrelやアクセシビリティ対応のテキストを設定できるようにしました。

  • Add noreferrerと Add nofollow: relにnoreferrerとnofollowをそれぞれ設定できます。別タブで開く設定にした時は自動でnoreferrerにチェックが入ります。外すことも可能です。
  • Accesibility link description: アクセシビリティ対応のテキスト設定にテキストを入れると、aria-labelとvisually-hiddenの要素を持つspanにテキストが入ります。デフォルトでは 「ブロック名 + link」のテキストが設定されます。

また@mthaichi さんの#2360 をベースに、deprecatedやAccesibility link description周りを微修正しました。
お1人は開発の方に見ていただきたいです。

スクリーンショットまたは動画

変更前 Before

スクリーンショット 2024-09-10 16 51 51

変更後 After

image

実装者の確認事項

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

  • 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
  • Files changed (変更ファイル)の内容は目視で確認したか?
  • readme.txt に変更内容は書いたか?
  • 本当にちゃんと確認をしたか?

プログラムの変更の場合

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

  • 書けそうなテストは書いたか?

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

新規、既存のOuterブロックともに以下を確認しました。

  • 編集画面でリンクツールバーのEdit linkからAdd noreferrer、Add nofollow、Accesibility link descriptionが追加できることを確認しました。
  • 編集画面で設定した relに noreferrer、nofollow、aria-labelとvisually-hiddenの要素を持つspanにAccesibility link descriptionに入力した内容がフロントエンドのソースで出力されていることを確認しました。
  • 編集画面でEdit linkを設定後、設定を削除したときに、relの noreferrer、nofollow、が消えていることを確認しました。また、aria-labelとvisually-hiddenの要素を持つspanに 「ブロック名 + link」のテキストが設定されます。
  • リンクだけ設定した場合はaria-labelとvisually-hiddenの要素を持つspanに 「ブロック名 + link」のテキストが設定されることを確認しました。

上記がOuter以外のリンク設定ができるブロック(グループやスライダーなど)では表示されないことを確認しました。

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

  • 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか?

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

実装者と同じ確認をしてください。
開発の方はコードの確認もお願いいたします。


レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

  • プルしたか?
  • ビルドしたか?
  • ビルドしたディレクトリは正しいか(別の開発環境のディレクトリを見ていないか)?
  • npm install したか?
  • composer install したか?
  • キャッシュをクリアして確認したか?

@mtdkei mtdkei changed the title [ リンクツールバー ] Edit linkを追加してaタグのアクセシビリティやSEOに対応(ここではOuterのみ設置) リンクツールバーにaタグのアクセシビリティやSEOに対応(ここではOuterのみ設置) Dec 3, 2024
@mtdkei mtdkei changed the title リンクツールバーにaタグのアクセシビリティやSEOに対応(ここではOuterのみ設置) リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定(ここではOuterのみ設置) Dec 3, 2024
@mtdkei mtdkei marked this pull request as ready for review December 4, 2024 01:13
@mtdkei mtdkei changed the title リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定(ここではOuterのみ設置) 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定(ここではOuterのみ設置) Dec 4, 2024
@mtdkei mtdkei changed the title 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定(ここではOuterのみ設置) 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ設置) Dec 4, 2024
@mtdkei mtdkei changed the title 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ設置) 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) Dec 4, 2024
@mtdkei mtdkei self-assigned this Dec 4, 2024
@MasayaMORIMOTO MasayaMORIMOTO changed the title 【確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) 【確認中】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) Dec 4, 2024
@MasayaMORIMOTO
Copy link
Collaborator

MasayaMORIMOTO commented Dec 4, 2024

動作確認しました。
記載通りの動作だと判断いたします。
変更された全てのコードの意図を汲んだ確認ではないため、
2人目の方は、開発の方、確認をお願いします。

@MasayaMORIMOTO MasayaMORIMOTO changed the title 【確認中】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) 【2人目確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) Dec 4, 2024
@mthaichi mthaichi changed the title 【2人目確認待ち】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) 【2人目確認中】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) Dec 4, 2024
Copy link
Contributor

@mthaichi mthaichi left a comment

Choose a reason for hiding this comment

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

@mtdkei ありがとうございます! お手間かけました🙏

@mthaichi mthaichi merged commit e696616 into develop Dec 6, 2024
14 checks passed
@mthaichi mthaichi deleted the feature/link-toolbar/empty-atag-accessibility branch December 6, 2024 08:23
@mthaichi mthaichi changed the title 【2人目確認中】リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) リンクツールバーにaタグのアクセシビリティやSEOに対応できるよう設定追加(ここではOuterのみ) Dec 6, 2024
mtdkei added a commit that referenced this pull request Dec 12, 2024
#2362 をマージした時にreadme.txtが古くなってしまったので、#2362 のブランチから追加されたChangelog「[ Add function ][ ballon ] Add default avatar.」のみを抽出し、一つ前の e696616 のreadmeに合わせました。
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.

3 participants