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

[5.9対応] 画像ブロックのスタイルのサンプル画像の高さが不足して表示される #955

Closed
shimotmk opened this issue Jan 14, 2022 · 9 comments
Assignees
Projects

Comments

@shimotmk
Copy link
Contributor

丸山調査中

画像ブロックのスタイルのサンプル画像の高さが不足して表示される

スクリーンショット 2022-01-11 午後6 47 59

Originally posted by @ChiakiKouno in #926 (comment)

@shimotmk shimotmk changed the title **丸山調査中** 画像ブロックのスタイルのサンプル画像の高さが不足して表示される Jan 14, 2022
@shimotmk shimotmk added this to To do in 5.9対応 via automation Jan 14, 2022
@shimotmk shimotmk moved this from To do to In progress in 5.9対応 Jan 14, 2022
@mthaichi mthaichi changed the title 画像ブロックのスタイルのサンプル画像の高さが不足して表示される [5.9対応] 画像ブロックのスタイルのサンプル画像の高さが不足して表示される Jan 14, 2022
@mthaichi
Copy link
Contributor

@goutetsuguma

src/extensions/core/image/style.scss

@mixin photoFrame 内の width: fit-content;
@mixin shadow 内の width: fit-content;

のいずれか1行を消すと正常に表示されるという腑に落ちていない途中なのですが、
いったん見ていただけますと助かりますー。

@goutetsuguma
Copy link
Contributor

goutetsuguma commented Jan 18, 2022

@shimotmk @mthaichi

VK Blocks Proを停止しても画像のスタイルのサンプルの高さが不足しているので、これはコアのブロックの不具合かなと思います。

▼私の方で確認したこと
src/extensions/core/image/style.scss の中に適当なプロパティと値を追加した場合もなぜか高さが出る。(width: fit-content;が原因ではない)
新規で固定ページを作成して画像をブロックを作成するとまた高さが不足してしまうので、何か変だなと思いVK Blocks Proを停止、停止しても画像のスタイルのサンプルの高さが不足していたのでコア側の不具合かな?

新規固定ページを追加 ‹ testsite — WordPress

@mthaichi
Copy link
Contributor

@goutetsuguma VK Blocks Pro 停止した時は同じ症状は確認できないですねー。私の方でももう少し確認してみますね。

@goutetsuguma
Copy link
Contributor

goutetsuguma commented Jan 18, 2022

@mthaichi

全然役に立たない情報かもしれませんが、、念の為書いておきます

私の場合はVK Blocks Pro を停止して(念の為vektor製品は全部停止)、固定ページを新規で作って、そこに画像ブロックを配置し(この時のスタイルのサンプルは高さが不足している状態)、保村して「固定ページを表示」を押して再び編集画面に戻ったとき、なぜかスタイルの高さが正常に表示されているという謎の現象がおきました、、、。
image-block

@mthaichi
Copy link
Contributor

mthaichi commented Jan 18, 2022

@goutetsuguma ありがとうございますー。
その時、CSSがキャッシュに入っていないという確認はされてますか。
CSSはキャッシュに入りやすいので、Shift押しながらリロードするなどしないと、再現したりしなかったりということになりそうです。されてたら失礼しました。いちおう念のため確認です。
あとでこちらでも再確認しますね。

@goutetsuguma
Copy link
Contributor

ありがとうございます!キャッシュ削除してませんでした、、 

(キャッシュ削除してからも私の環境ではVK Blocks Pro を停止していても高さが不足していたりするので、、また何か分かりましたらこちらに書き込みます🙇‍♂️)

@mthaichi
Copy link
Contributor

mthaichi commented Jan 19, 2022

@goutetsuguma たしかにこれはコアの問題っぽいですね。
WordPressをゼロベースでインストールして、5.9RC3にアップデートしただけの環境でも再現できました。
失礼しました。

ただ、再読み込みすると正常に表示されたりしなかったりで、どういうときにこうなるのかわからないですね・・・。

@ChiakiKouno
Copy link
Member

ChiakiKouno commented Jan 20, 2022

想像以上にややこしいバグを上げてしまってすみません、、
もう一度確認してみました。
Chromeの検証画面にして、スマホの画面サイズを適用した後に症状が出るっぽいようでした。なんとも言えないなんだこれは。。

「検証画面でスマホ表示する」ことがない人にはあまり関係ない症状なのかな、と思ったり。。

2022-01-20.12.33.22.mov

@shimotmk
Copy link
Contributor Author

コアのバグということでcloseします
vk-blocksの影響があったらreopenしてください

@shimotmk shimotmk moved this from In progress to Done in 5.9対応 Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

No branches or pull requests

4 participants