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

【確認待ち】Outerブロックカラーパレット対応 #1032

Closed

Conversation

mthaichi
Copy link
Contributor

@mthaichi mthaichi commented Feb 24, 2022

大幅な構造変更をかけましたので、入念なるチェックをお願いできればと思います。
開発メンバー以外の方でも確認できる環境をお持ちであれば、動作チェックしていただけると嬉しいです。(ご無理なく)
特に以下の点についてご確認ください。

  • 背景色でキーカラー追従されるか
  • 枠線色がキーカラー追従されるか
  • 上下区切り色がキーカラー追従されるか
  • 枠線色を指定しても中のテキスト色に影響はないか
  • 前バージョンのブロックが壊れずに動作するか
  • 背景画像を各メディアタイプに設定して、きちんと表示されるか
  • 枠線の色・スタイルは反映されるか
  • 背景色透過が適切に動作するか
  • ブレイクポイントは適切に設定されているか 【残り いしかわさん担当】 レスポンシブ処理の調整 #763
  • Radiusの値がマイナスのブロックをきちんとリカバリーできているか。(Radiusの最低値を0に変更しました)

以前からブランチを変更しております。

# Conflicts:
#	inc/vk-blocks/languages/vk-blocks-js.pot
#	inc/vk-blocks/languages/vk-blocks.pot
# Conflicts:
#	inc/vk-blocks/languages/vk-blocks.pot
…r/custom-color-palette

# Conflicts:
#	inc/vk-blocks/languages/vk-blocks-js.pot
#	src/blocks/_pro/outer/index.php
fix: ブレイクポイン値の変更
@mthaichi mthaichi changed the title 【確認待ち】Outerブロックカラーパレット対応 【調整中】Outerブロックカラーパレット対応 Feb 24, 2022
@mthaichi mthaichi marked this pull request as draft February 24, 2022 10:07
@mthaichi mthaichi marked this pull request as ready for review February 24, 2022 10:08
@mthaichi mthaichi changed the title 【調整中】Outerブロックカラーパレット対応 【確認待ち】Outerブロックカラーパレット対応 Feb 24, 2022
@mthaichi mthaichi changed the base branch from master to develop February 24, 2022 10:17
@shimotmk
Copy link
Contributor

@mthaichi
見かけ上は対応出来ているように思います

ただ実装でenque-front.jsを使っていますが、enque-frontはフロントエンドのみのjsのためスタイルを変更することには使わないほうが良いと思います
保存されているHTMLと表示されるHTMLが異なると今後変更があった時に大変なことになりそうです

Outerブロックで機能追加したい箇所もあるので一旦カスタムカラー対応 私の方で引き取っても良いでしょうか?

@mthaichi
Copy link
Contributor Author

mthaichi commented Feb 25, 2022

@mthaichi 見かけ上は対応出来ているように思います

ただ実装でenque-front.jsを使っていますが、enque-frontはフロントエンドのみのjsのためスタイルを変更することには使わないほうが良いと思います 保存されているHTMLと表示されるHTMLが異なると今後変更があった時に大変なことになりそうです

Outerブロックで機能追加したい箇所もあるので一旦カスタムカラー対応 私の方で引き取っても良いでしょうか?

@shimotmk これは前にお話した下記のようCurrentColorを使った場合、CSS自体の枠線スタイルがgrooveやridgeのときだけ適用されないことの対応でした。JSはブロックの動作を大きく左右するものではないですが、has-border系のクラスができたんでしたっけ。それで対応できれば、そちらがいいですね。

どうぞいったん引き取ってくださいー。

<style>
div {
		 border: 10px groove currentColor;
		 color: red;
}		
</style>

<div>Test</div>

@shimotmk shimotmk closed this Feb 25, 2022
@shimotmk shimotmk deleted the fix/outer/custom-color-pallete-bgcontainer-style branch February 25, 2022 02:43
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.

8 participants