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

block.js から vk-blocks-build-css の記述を削除 #2392

Merged
merged 3 commits into from
Jan 5, 2025

Conversation

drill-lancer
Copy link
Member

@drill-lancer drill-lancer commented Dec 27, 2024

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

#2265

どういう変更をしたか?

vk-blocks-build-css の読み込み優先度を下げるため block.js から vk-blocks-build-css の記述を削除しました。

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

変更前 Before

before

変更後 After

after

実装者の確認事項

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

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

プログラムの変更の場合

  • 書けそうなテストは書いたか?
    ⇒ CSS の読み込み順を変更したのみなので省略

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

<!-- wp:vk-blocks/button {"buttonType":"1","blockId":"16e89875-5a01-4218-8644-766839036498"} -->
<div class="wp-block-vk-blocks-button vk_button vk_button-color-custom vk_button-align-left"><a class="vk_button_link btn has-text-color is-style-outline has-vk-color-primary-color btn-md" role="button" aria-pressed="true" rel="noopener"><div class="vk_button_link_caption"><span class="vk_button_link_txt">背景なし Primary色 テストボタン</span></div></a></div>
<!-- /wp:vk-blocks/button -->

<!-- wp:vk-blocks/accordion {"className":"is-style-vk_accordion-background-color-border"} -->
<div class="wp-block-vk-blocks-accordion vk_accordion is-style-vk_accordion-background-color-border" data-initial-state="close" data-initial-state-mobile="" data-initial-state-tablet="" data-initial-state-desktop="" data-device-specific="false"><!-- wp:vk-blocks/accordion-trigger -->
<div class="wp-block-vk-blocks-accordion-trigger vk_accordion-trigger"><!-- wp:paragraph -->
<p>アコーディオンテスト</p>
<!-- /wp:paragraph --><span class="vk_accordion-toggle vk_accordion-toggle-close"></span></div>
<!-- /wp:vk-blocks/accordion-trigger -->

<!-- wp:vk-blocks/accordion-target -->
<div class="wp-block-vk-blocks-accordion-target vk_accordion-target"><!-- wp:paragraph -->
<p>アコーディオンテスト</p>
<!-- /wp:paragraph --></div>
<!-- /wp:vk-blocks/accordion-target --></div>
<!-- /wp:vk-blocks/accordion -->
  • 上記コードををコードエディタで貼り付けた
  • テーマを X-T9 に変更
  • 分割読み込みを無効化

develop ブランチだとボタンブロックが真っ白になるがこのブランチだと正常に表示される

上記を確認お願いします。

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

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

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

<!-- wp:vk-blocks/button {"buttonType":"1","blockId":"16e89875-5a01-4218-8644-766839036498"} -->
<div class="wp-block-vk-blocks-button vk_button vk_button-color-custom vk_button-align-left"><a class="vk_button_link btn has-text-color is-style-outline has-vk-color-primary-color btn-md" role="button" aria-pressed="true" rel="noopener"><div class="vk_button_link_caption"><span class="vk_button_link_txt">背景なし Primary色 テストボタン</span></div></a></div>
<!-- /wp:vk-blocks/button -->

<!-- wp:vk-blocks/accordion {"className":"is-style-vk_accordion-background-color-border"} -->
<div class="wp-block-vk-blocks-accordion vk_accordion is-style-vk_accordion-background-color-border" data-initial-state="close" data-initial-state-mobile="" data-initial-state-tablet="" data-initial-state-desktop="" data-device-specific="false"><!-- wp:vk-blocks/accordion-trigger -->
<div class="wp-block-vk-blocks-accordion-trigger vk_accordion-trigger"><!-- wp:paragraph -->
<p>アコーディオンテスト</p>
<!-- /wp:paragraph --><span class="vk_accordion-toggle vk_accordion-toggle-close"></span></div>
<!-- /wp:vk-blocks/accordion-trigger -->

<!-- wp:vk-blocks/accordion-target -->
<div class="wp-block-vk-blocks-accordion-target vk_accordion-target"><!-- wp:paragraph -->
<p>アコーディオンテスト</p>
<!-- /wp:paragraph --></div>
<!-- /wp:vk-blocks/accordion-target --></div>
<!-- /wp:vk-blocks/accordion -->
  • 上記コードををコードエディタで貼り付けた
  • テーマを X-T9 に変更
  • 分割読み込みを無効化

develop ブランチだとボタンブロックが真っ白になるがこのブランチだと正常に表示される

上記を確認お願いします。


レビュワー向け

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

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

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

@drill-lancer drill-lancer reopened this Dec 27, 2024
@drill-lancer drill-lancer changed the base branch from master to develop December 27, 2024 04:51
@drill-lancer drill-lancer reopened this Dec 27, 2024
@drill-lancer drill-lancer self-assigned this Dec 27, 2024
@drill-lancer drill-lancer marked this pull request as ready for review December 27, 2024 05:00
@akito-38 akito-38 changed the title 【確認待ち】block.js から vk-blocks-build-css の記述を削除 【確認中】block.js から vk-blocks-build-css の記述を削除 Dec 27, 2024
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.

@drill-lancer
指定いただいた方法にて確認いたしました。
問題無いと思いますので、二人目の方確認お願いいたします。

@akito-38 akito-38 changed the title 【確認中】block.js から vk-blocks-build-css の記述を削除 【二人目確認待ち】block.js から vk-blocks-build-css の記述を削除 Dec 27, 2024
Copy link
Contributor

@mtdkei mtdkei left a comment

Choose a reason for hiding this comment

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

ありがとうございます。アコーディオンブロック以外でも、Files changedにあるブロックを全て確認しました。
なお、以前issueのコメントに書いた通り、スライダー系は本件のissueのような現象が起きなかったのですが、書き方が揃っている方がいいのかもしれませんのでこのままマージいたします。

@mtdkei mtdkei changed the title 【二人目確認待ち】block.js から vk-blocks-build-css の記述を削除 block.js から vk-blocks-build-css の記述を削除 Jan 5, 2025
@mtdkei mtdkei merged commit 0446770 into develop Jan 5, 2025
13 checks passed
@mtdkei mtdkei deleted the fix/vk-blocks-build-css branch January 5, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants