-
Notifications
You must be signed in to change notification settings - Fork 10
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
Upgrade frontend to Next.js 13.x #319
Conversation
hey @ferostar ok for me 👍 |
Please test this PR on: https://v2-staging-319-dot-fuji-306908.ey.r.appspot.com |
@brozorec just realized something went missing in the UI after upgrading, I'll let you know once I'm done |
Tried all solutions provided so far:
It doesn't work. Only setting |
I always noticed that we have a FOUT issue (that the default font appears for a bit before switching to our font), so the whole screen changes size. Known issue.
Didn't want to spend much time on it, but I saw that in
next 13.x
they added a new submodule to preload fonts that work particularly well with Google Fonts, as the one we're using.I quickly upgraded to
13
, solved a few problems, and changed the way we load fonts. There's no FOUT anymore, and it seems everything looks good. Again, I spent only a couple of minutes trying this, so it could be just a starting point, but the impact is significant!@brozorec care to have a look?