vektor-inc / vk-blocks-pro

11 stars 3 forks source link

[ core/table ] スクロールヒントを設定している状態で「非表示設定」を設定した時に非表示になるように修正 #2249

Closed mtdkei closed 5 days ago

mtdkei commented 5 days ago

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

2248

どういう変更をしたか?

is-style-vk-*-scrollable クラスが適用されているブロックにvk_hiddenなどの非表示クラスが含まれている場合、それらのクラスをすべてvk-scroll-hintに適用するように変更。

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

変更前 Before

スクリーンショット 2024-10-01 9 43 48

変更後 After

スクリーンショット 2024-10-01 9 44 28

実装者の確認事項

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

プログラムの変更の場合

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

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

  1. スクロールメッセージ付きの横スクロール設定をしたテーブルブロックを追加し、非表示設定を複数設定し保存。
  2. 非表示設定に伴い、スクロールメッセージも非表示になることを確認。
  3. 編集画面で非表示設定の項目を増減したときでも、フロントエンドのスクロールメッセージも非表示で設定した通りになることを確認。
  4. 非表示設定を全てOFFにしたときに、tableとともにスクロールメッセージが消えないことを確認。

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

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

実装者と同じ確認を行ってください。


レビュワー向け

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

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

goutetsuguma commented 5 days ago

調整ありがとうございます!動作確認いろいろしまして問題なさそうでした! 2人目確認待ちにします

sysbird commented 5 days ago

対応ありがとうございます、 確認しました マージします〜