Introduce an "enhanced" theme selector for user settings#5251
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
dbe5cbd to
dff0992
Compare
|
Looks very nice! Just curious; do you know why cypress cant save changes? I could do it. |
nooo idea 😆 |
Doesn't work for me. I can press the save-button if I change the theme, but if I do not edit it the save button is deactivated :'( The button is also deactivated if I change it and then change back before I try to save Hmm upon further inspection what happened to me might be on purpose, in contrast to what happens to cypress |
|
This peaked my curiosity, so I spent some time testing it. Seems that |
falbru
left a comment
There was a problem hiding this comment.
Looks really good! Great job 😍
Yes it's impossible to reproduce locally .. Thanks a lot for looking into it though! ❤️ |
|
One step closer to looking better than TIHLDEs website😆 |
dff0992 to
7f77937
Compare
|
lets go 🔥 |
Its time😝 |
Glad to see this is alive 🤘 |
|
Context for the thread above: SSR/hydration was reinitializing the profile form, which reset Final Form’s pristine state after field edits, so the "Lagre endringer" submit button stayed disabled in CI even though the Cypress flow worked locally. Impossible to reproduce locally and CI Cypress run is flaky, giving unreliable result when nothing changed😩 |
😩 i was before my time then - would've needed claude to resolve that bitch |
Description
Not a fan of the radio buttons
Result
Testing
Changing themes works fine, and I've tested that changing it through the navbar updates the form (unlike before).
Resolves ABA-1212