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

ユニバーサルUI以外のUIで一部のスタイルが適用されない #9302

Closed
tamaina opened this issue Dec 10, 2022 · 26 comments · Fixed by #9313 or #9349
Closed

ユニバーサルUI以外のUIで一部のスタイルが適用されない #9302

tamaina opened this issue Dec 10, 2022 · 26 comments · Fixed by #9313 or #9349
Labels
⚠️bug? This might be a bug

Comments

@tamaina
Copy link
Member

tamaina commented Dec 10, 2022

image

image

Vite v4が関係しているかも?
どのスタイルを読み込めていないのか把握できていない

Misskey version: v12.120.0-alpha.14
Your OS: macOS Ventura
Your browser: Chrome, Firefox, Safari

@tamaina tamaina added the ⚠️bug? This might be a bug label Dec 10, 2022
@syuilo
Copy link
Member

syuilo commented Dec 10, 2022

ローカルでは再現しなかったわ
productionモードでビルドするとそうなるのかな

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

@syuilo これと関係があるかわからないけどyarn.lockが正しいか確認してみてほしい

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

developmentモードでもこうなるような…

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

うん、NODE_ENV=developmentでもぶっ壊れる

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

update depsはdevelopにプッシュした

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

https://vitejs.dev/guide/migration.html#importing-css-as-a-string

順当に考えると原因これだろうなぁ

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

link(rel='preload' href=`/assets/${href}` as='style')

style.scssが読み込めていないのかなと思ってここをrel='stylesheet'にしてみたけどダメだった

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

style.scssが読み込めていないのかなと思ってここをrel='stylesheet'にしてみたけどダメだった

これやるとユニバーサルUIもぶっ壊れる

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

iOS Safariだと問答無用で崩壊した

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

これimport baseStyle from '@/style.scss?inline';でCSSをstringで取得して(baseStyleはstring)適当にstyle作って突っ込めってことなんか?

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

そもそも使い方がおかしかった説(だからと言ってどうやってimportしようか悩むところではある)

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

visitorで最初に一瞬だけ見えるこれってなんだろうか

image

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

visitorで最初に一瞬だけ見えるこれってなんだろうか

MkLoadingっぽい?

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

ほう、MkLoadingやMkPageHeaderのCSSも欠けてる

ということはcomponents/globalもダメそう

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

@/style.scssはcommon.vueで@use "../../style.scss";することにする

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

これimport baseStyle from '@/style.scss?inline';でCSSをstringで取得して(baseStyleはstring)適当にstyle作って突っ込めってことなんか?

Stackblitzのサンプルではimport './style.css'をガッツリ使っているのでそんなことはなさそう

image

@tamaina
Copy link
Member Author

tamaina commented Dec 10, 2022

ローカルでは再現しなかったわ

これ怪奇現象なんだけど(Windowsだから再現しないの??)

@CyberRex0
Copy link
Contributor

macOSのSafariだと未ログイン時のトップページが崩れるだけで他は問題なかった

@syuilo
Copy link
Member

syuilo commented Dec 11, 2022

ローカルでは再現しなかったわ

これ怪奇現象なんだけど(Windowsだから再現しないの??)

あーデッキで再現した

@tamaina
Copy link
Member Author

tamaina commented Dec 12, 2022

Minimum Reproductionから逃げてるんだけど誰か作って欲しい()

@tamaina
Copy link
Member Author

tamaina commented Dec 12, 2022

Vite v4のせいではない説が濃厚になってきた

これのせい?
2f9de0c

@tamaina
Copy link
Member Author

tamaina commented Dec 12, 2022

Vite v4のせいではない説が濃厚になってきた

これのせい? 2f9de0c

これも違うっぽい

@tamaina
Copy link
Member Author

tamaina commented Dec 12, 2022

コミット1つずつおってくか

2067180 →動く

@tamaina
Copy link
Member Author

tamaina commented Dec 12, 2022

c5213dd →動かない ←犯人

@tamaina
Copy link
Member Author

tamaina commented Dec 18, 2022

rollupOptions.outputを書けばreproductするかもしれないけど書くまでが面倒だわね

@tamaina
Copy link
Member Author

tamaina commented Dec 18, 2022

https://vitejs.dev/config/build-options.html#build-modulepreload

it is necessary to manually import the polyfill in your custom entry:

I have overlooked this until now, this could be part of the cause

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
3 participants