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対応] wp_add_inline_styleで追加しているcssがプレビューで反映されない #959

Closed
shimotmk opened this issue Jan 14, 2022 · 6 comments · Fixed by #975

Comments

@shimotmk
Copy link
Contributor

https://twitter.com/tetsuaki_hamano/status/1481494203385663490
FontAwesomeやvk-blocksの管理画面で設定できる吹き出しの線幅の設定がプレビューで反映されない

@mthaichi mthaichi changed the title wp_add_inline_styleで追加しているcssがプレビューで反映されない [5.9対応] wp_add_inline_styleで追加しているcssがプレビューで反映されない Jan 14, 2022
@kurudrive
Copy link
Member

kurudrive commented Jan 14, 2022

@mthaichi
Copy link
Contributor

これ見ます。

@shimotmk
Copy link
Contributor Author

@mthaichi
Copy link
Contributor

mthaichi commented Jan 26, 2022

@kurudrive @shimotmk @drill-lancer
fontAwesomeがプレビューで出力されない件ですが、loos/SWELL#84 にの最後にありますように .wp-block というセレクタ が CSSの中で定義されていないと読み込まれないようです。

#_iframehack.wp-block {
    color: inherit;
}

を 読み込まれるCSSに入れるしかなさそうです。
vektor-inc/font-awesome-versions パッケージの versions/ 内の各CSSの先頭に、上記CSSを埋め込む形が一番スマートな方法だと判断してますが、fontawesomeのCSS自体に上記コードを埋め込むのはライセンス的には妥当でしょうか。また、このCSSがなんらかのアップデートで上書きされることはないでしょうか。

また、現状ブロックエディタの時は wp_enqueue_style でfont-awesomeを読み込んでいますが、add_editor_style で読み込むのが望ましいとのことで、そのように変更したいと思います。

ここまで自分の開発環境で動作するのは確認しました。

上記 vektor-inc/font-awesome-versions パッケージを変更する形になり、影響範囲が大きそうなので、確認をさせてください。もし懸念事項などあれば、教えてもらえると嬉しいです。

あと、ExUnitはどうする問題もありますね・・・。

@shimotmk
Copy link
Contributor Author

@mthaichi
ここでも言われているように .wp-blockというセレクタを入れるしかなさそうですね。
ただ他にも色々issueが上がっているので.wp-blockがなくても、いずれ出来る方法ができそうですが😅

Font AwesomeのコードはMIT Licenseなので改変大丈夫だと思います。
https://fontawesome.com/license/free

@shimotmk
Copy link
Contributor Author

FontAwesomeは#1016 で対応されたようなのでcloseします!

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