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

【確認中】画像ブロックのスタイルのサンプルに流体シェイプのマスクのsvgがかからない問題調整 #972

Merged
merged 2 commits into from
Jan 24, 2022

Conversation

shimotmk
Copy link
Contributor

@shimotmk shimotmk commented Jan 21, 2022

fix : #966
5.8、5.9でスタイルサンプルとモバイル、タブレットのプレビューで問題ないか確認お願いします。
コアの画像ブロックに依存するのでwp-edit-blocksを追加しました。

参考:WordPress/gutenberg#33212 (comment)

--vk_image-mask-waveはコアの画像ブロックに依存するのでwp-edit-blocksを追加
参考 : WordPress/gutenberg#33212 (comment)
@shimotmk shimotmk changed the title 画像ブロックのスタイルのサンプルに流体シェイプのマスクのsvgがかからない問題調整 【確認待ち】画像ブロックのスタイルのサンプルに流体シェイプのマスクのsvgがかからない問題調整 Jan 21, 2022
@sysbird sysbird changed the title 【確認待ち】画像ブロックのスタイルのサンプルに流体シェイプのマスクのsvgがかからない問題調整 【確認中】画像ブロックのスタイルのサンプルに流体シェイプのマスクのsvgがかからない問題調整 Jan 21, 2022
@sysbird
Copy link
Member

sysbird commented Jan 21, 2022

編集画面の右パネルのこれ

shape

@sysbird
Copy link
Member

sysbird commented Jan 21, 2022

@shimotmk
反映されてないように見えます
シェイプのサンプルはどれも四角いままです
私のローカル環境は、WordPress 5.9-RC3-52601 で、テーマは Ligntning系、twenty系と試しました

@shimotmk
Copy link
Contributor Author

@sysbird
あれ??ホントですか?
私はちゃんとプレビューされてます、、

twenty-twenty-one

twenty-twenty-one

Ligntning

Ligntning

@sysbird
Copy link
Member

sysbird commented Jan 21, 2022

@shimotmk
Chrome で確認したらOKです!

Firefoxでは四角いままで、よくみたらフォトフレームの傾きなどもスタイルが当たってないです、キャッシュはクリアしました。これは…どうしましょうか、Firefoxは最近ネットができなくなったりダメダメなんですよ〜

@shimotmk
Copy link
Contributor Author

@sysbird
ありがとうございます!
確かにFirefox当たってないですね。編集画面の環境依存は辛いな

一旦来週までこのissuesの動向を見つつFirefox対応の調査してみます。
WordPress/gutenberg#33212 (comment)

@shimotmk
Copy link
Contributor Author

@sysbird
色々調査したのですがFirefoxのプレビューはコアのバグっぽいので、ここではchromeで調整してコアの修正を待つことにしたいと思います。

コアのissue
WordPress/gutenberg#37961
WordPress/gutenberg#29436

@sysbird
Copy link
Member

sysbird commented Jan 24, 2022

@shimotmk
ありがとうございます
フロントが見えてて、ブラウザ依存なので当面はよいかと
モバイルプレビューを確認してマージしておきます!

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.

2 participants