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

【二人目確認待ち】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 #2447

Open
wants to merge 22 commits into
base: develop
Choose a base branch
from

Conversation

mtdkei
Copy link
Contributor

@mtdkei mtdkei commented Feb 6, 2025

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

#2446

どういう変更をしたか?

修正対象 原因 修正方法
投稿リスト / 投稿リストスライダー aタグ の preventDefault() が iframe 内で効いてしまう disableLinks() の適用範囲を調整
投稿リスト / 投稿リストスライダー vk-components.css が iframe内で適用されない enqueue_block_assetsからvk-components.cssを読みこませるように変更(またfunctionsが2つ以上になったため、共通処理を整理)
タブ querySelector() が iframe を考慮していなかった iframe.contentWindow.document.querySelector() を使用
目次 OPEN / CLOSE ボタンが横に伸びる style.maxWidth を指定
スライダー Swiper の初期化が iframe 内で動作しない iframe.contentWindow.document を参照して Swiper を初期化

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

変更前 Before

投稿リスト
スクリーンショット 2025-02-07 10 24 06

投稿リストスライダー
スクリーンショット 2025-02-07 10 24 22

タブ
画面収録 2025-02-07 10 16 17

目次
スクリーンショット 2025-02-07 10 15 52

スライダー
スクリーンショット 2025-02-07 10 15 43

変更後 After

投稿リスト
スクリーンショット 2025-02-07 13 05 36

投稿リストスライダー
スクリーンショット 2025-02-07 13 05 42

タブ
画面収録 2025-02-07 10 28 36

目次
スクリーンショット 2025-02-07 10 29 07

スライダー
画面収録 2025-02-07 10 29 35

実装者の確認事項

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

  • 複数の意図の変更 ( 機能の不具合修正 + 別の機能追加など ) を含んでいないか?
  • Files changed (変更ファイル)の内容は目視で確認したか?
  • readme.txt に変更内容は書いたか? → apiVersion3のリリース前のためスキップ
  • readme.txt に記載の変更内容はエンドユーザーが見て変更の概要がわかるように書かれているか? → apiVersion3のリリース前のためスキップ
  • 本当にちゃんと確認をしたか?

プログラムの変更の場合

  • 書けそうなテストは書いたか? →スキップ

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

developでかつX-T9やTT5の時は以下の挙動をしていたブロックがこのブランチでは直っていたことを確認しました。
分割読み込みのON/OFFも確認済みです。

  • 投稿リスト
    • aタグを無効にしていたのが効いてしまう。
    • vk-components.cssが読み込まれず、colの大きさや右上カテゴリーラベルのデザインが適用されない。
  • 投稿リストスライダー
    • aタグを無効にしていたのが効いてしまう。
    • vk-components.cssが読み込まれず、colの大きさや右上カテゴリーラベルのデザインが適用されない。
  • タブ
    • 編集画面でタブラベルをクリックしても、タブアイテムが切り替わらない。
  • 目次
    • OPEN / CLOSE ボタンが横に伸びる。
  • スライダー
    • プレビュー(スライド)モードでスライダーにならない。

元々developでも問題がなかったLightiningで確認した場合でもこのブランチによる悪影響がないことを確認しました。

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

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

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

実装者と同じ確認を行ってください。
特に/inc/vk-blocks/load-vk-components.phpの読み込み周りは複数のfunctionsが入るつくりではなかったのでよろしければコードをみていただけたら嬉しいです。


レビュワー向け

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

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

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

@mtdkei mtdkei changed the base branch from master to develop February 6, 2025 05:48
@mtdkei mtdkei changed the title Fix/block editor/api ver 3 develop かつ VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 Feb 6, 2025
@mtdkei mtdkei marked this pull request as ready for review February 7, 2025 06:02
@mtdkei mtdkei changed the title develop かつ VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 【確認待ち】develop かつ VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 Feb 7, 2025
@mtdkei mtdkei self-assigned this Feb 7, 2025
@mtdkei mtdkei changed the title 【確認待ち】develop かつ VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 【確認待ち】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 Feb 7, 2025
@akito-38 akito-38 changed the title 【確認待ち】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 【確認中】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 Feb 9, 2025
Copy link
Contributor

@akito-38 akito-38 left a comment

Choose a reason for hiding this comment

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

@mtdkei

実装者の確認方法と同様に、T9やTT5、Lightiningで対象ブロックを確認し、問題無く動作していました。
また、コードも確認しましたが、問題無いように思います。
二人目の方、確認お願いいたします。

@akito-38 akito-38 changed the title 【確認中】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 【二人目確認待ち】developブランチで VK製テーマ(X-T9) 以外のテーマでは 編集画面の挙動がおかしいブロックがあるのを修正 Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants