vektor-inc / vk-blocks-pro

11 stars 3 forks source link

[ 固定表示 ]「下部からの固定表示」という項目を追加しました #2230

Closed mtdkei closed 1 week ago

mtdkei commented 1 week ago

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

2226

どういう変更をしたか?

「上部からの固定位置」だけでなく、「下部からの固定位置」を選択できるように追加しました。

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

変更前 Before

スクリーンショット 2024-09-26 12 35 33

変更後 After

スクリーンショット 2024-09-26 12 34 47 スクリーンショット 2024-09-26 12 40 13

実装者の確認事項

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

プログラムの変更の場合

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

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

  1. 編集画面で固定表示ブロックを作成。
  2. 「固定位置の基準」で「上部」または「下部」を選択し、適当な数値と単位を設定。
  3. 保存し、フロントエンドで以下が適用されているかを確認。
    • クラス .vk_fixed-display-position-from-top もしくは .vk_fixed-display-position-from-bottom が適用されることを確認。
    • 「上部からの固定位置」の時はstyle="top: ◯◯svh"のようになり、「下部からの固定位置」の時はstyle="bottom: ◯◯svh"のようになることを確認。

また、 https://patterns.vektor-inc.co.jp/vk-patterns/18641/ のパターン内で「固定位置: 右」の設定がされている固定表示ブロックで以下を確認しました。

  1. 「固定位置の基準」で「下部」を選択し、下部からの固定位置で「0」を設定。
  2. カスタムCSSの以下の記述を削除。
    bottom: 0;
    top: unset !important;
  3. フロントエンドで、カスタムCSSがない状態でも bottom: 0; になることを確認。

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

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

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


レビュワー向け

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

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

sysbird commented 1 week ago

@mtdkei 確認したところ、いいかんじですね! さっそく対応ありがとうございます、固定表示ブロックますます便利になります

goutetsuguma commented 1 week ago

調整ありがとうございます!問題なさそうでしたのでマージします