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

【確認中】スライダーアイテムをカスタムカラーパレットに対応 #925

Merged
merged 18 commits into from
Jan 27, 2022

Conversation

shimotmk
Copy link
Contributor

@shimotmk shimotmk commented Dec 22, 2021

fix: #765 #763

スライダーブロックのカスタムカラーに対応しました。

  • 背景色がカラーパレット、カスタムカラー追従するか
  • 透過設定が以前と同じように動作しているか
  • 背景画像サイズが設定されているか

確認お願いします。

実装方法メモ

基本的にはコアのcoverブロックを参考
透過設定のcss

リリース時に相談したいこと

実際にクローンして確認していただきたいだのですがHTMLが変わります。
特に背景画像がない時に変わるので以下にHTMLを貼っておきます。
master

<div class="wp-block-vk-blocks-slider-item vk_slider_item vk_slider_item-〇〇">
  <style>
    .vk_slider_item-〇〇{background: linear-gradient(rgba(130, 36, 227, 0.5), rgba(130, 36, 227, 0.5)); background-repeat:repeat!important; background-position:center!important;}
  </style>
  <div class="vk_slider_item_container container">
    <p>スライダーコンテンツ</p>
  </div>
</div>

このブランチ

<div class="wp-block-vk-blocks-slider-item vk_slider_item vk_slider_item-〇〇 has-background-color has-vk-color-primary-vivid-background-color has-background-dim has-background-dim-5">
  <div class="vk_slider_item-background-area"></div>
  <div class="vk_slider_item_container container">
    <p>スライダーコンテンツ新</p>
  </div>
</div>

<div class="vk_slider_item-background-area"></div>が入ります。

以前のアコーディオンブロックのようにリリースする前に事前告知したほうが良いですか。問題ないですかね

@drill-lancer

This comment has been minimized.

@shimotmk

This comment has been minimized.

@drill-lancer

This comment has been minimized.

@shimotmk shimotmk changed the title 【調整中】スライダーアイテムをカスタムカラーパレットに対応 【コンポーネント調整中】スライダーアイテムをカスタムカラーパレットに対応 Dec 24, 2021
@shimotmk shimotmk changed the title 【コンポーネント調整中】スライダーアイテムをカスタムカラーパレットに対応 【調整中】スライダーアイテムをカスタムカラーパレットに対応 Jan 21, 2022
@shimotmk shimotmk changed the title 【調整中】スライダーアイテムをカスタムカラーパレットに対応 【確認待ち】スライダーアイテムをカスタムカラーパレットに対応 Jan 25, 2022
@shimotmk shimotmk marked this pull request as ready for review January 25, 2022 04:45
@sysbird sysbird changed the title 【確認待ち】スライダーアイテムをカスタムカラーパレットに対応 【確認中】スライダーアイテムをカスタムカラーパレットに対応 Jan 27, 2022
@sysbird
Copy link
Member

sysbird commented Jan 27, 2022

<div class="vk_slider_item-background-area"></div>が入ります。

以前のアコーディオンブロックのようにリリースする前に事前告知したほうが良いですか。問題ないですかね

そういえば、枠線ブロックの際は気づかず反映してしまいました…
ブロックのリカバリーもなく、ユーザー側の対応が発生しないのでいらない気がします

@kurudrive
ご判断お願いできましたらば〜

* develop:
  [ Change version ] 1.22.3
  fix : 見出しブロックが全幅にならない不具合修正
  [ Change version ] 1.22.2
  [ 不具合修正 ][ 選択投稿ブロック ][ vk posts ] 投稿idが正しく反映されない(結果リンクターゲットのblankが付かない)不具合を修正

# Conflicts:
#	inc/vk-blocks/languages/vk-blocks.pot
#	readme.txt
@sysbird sysbird merged commit 1d6bffd into develop Jan 27, 2022
@sysbird
Copy link
Member

sysbird commented Jan 27, 2022

旧ブロック / 新ブロック × WP5.8 / 5.9で確認しました

@shimotmk
Copy link
Contributor Author

事前告知について
HTMLの階層が変わることやクラス名が変更の場合は事前告知が必要だが今回のケースはdivが挟まるだけなので今回は無しということで落ち着きました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants