-
Notifications
You must be signed in to change notification settings - Fork 4
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
[ 投稿リスト / 投稿リストスライダー / タブ /スライダー ] 編集画面の環境がiframeであるかないか関わらずエラーが出たり機能が安定して動作するよう修正 #2458
Conversation
こちらの手順を試したのですが、"apiVersion": 3 を 2 に変更して npm run build を実行すると、再度 3 に書き戻されてしまい、「変更前 Before」の状態を確認することができませんでした(現状では変更後の状態になっているようです)。 "apiVersion": 2 のままビルドする方法があれば教えていただけますでしょうか? すみませんが、ご確認よろしくお願いします。 |
@goutetsuguma |
@goutetsuguma |
@mtdkei 少し気になるところは、もともとわたしのLocalが重たすぎるのでそのせいかもしれませんが、「スライダー」ブロックの「編集モード」を「プレビュー(スライド)」から「編集(縦積み配置)」に切り替えた時に、切り替わるまでに数秒かかるようでした。たまにページが固まることもあります。 いったん2人め確認待ちにします。 |
@goutetsuguma |
モード切り替えの時に切り替えが遅い現象について対応しました。 |
@mtdkei 調整ありがとうございます! |
@mtdkei |
@akito-38 |
@mtdkei ブランチ:fix/apiVersion3/iframe,develop default.mp4 |
@akito-38 |
@mtdkei ではこのプルリクは問題ないと思いますので承認しておきます。 |
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
#2457
どういう変更をしたか?
iframe.contentWindow.document
または通常のdocument
を適切に使用するロジックを実装しました。また、MutationObserver の監視対象も iframe 環境か否かをそれぞれ考慮するよう改善しました。これにより、iframe 環境の有無に関わらずでも安定してスライダーとタブ切り替え機能が動作するようになりました。対象ブロック
スクリーンショットまたは動画
変更前 Before
投稿リスト

投稿リストスライダー

タブ

スライダー

変更後 After
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
変更内容について何を確認したか、どういう方法で確認をしたかなど
git checkout 8454579
をし、inc/vk-blocks/build/blocks/alert/block.json
で"apiVersion": 3
の数字を2
にしてnpm run build
を実行。git checkout fix/apiVersion3/iframe
とnpm run build
を実行。"apiVersion": 2
にしていた数字を3
にしてnpm run build
を実行。レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
git checkout 8454579
をし、inc/vk-blocks/build/blocks/alert/block.json
で"apiVersion": 3
の数字を2
にしてnpm run build
を実行してください。git checkout fix/apiVersion3/iframe
とnpm run build
を実行してください。"apiVersion": 2
にしていた数字を3
にしてnpm run build
を実行してください。その他気になることがあれば適宜ご確認ください。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。