ost-cas-fee-adv-22-23 / app-thierry-und-simon

https://app-thierry-und-simon.vercel.app
Apache License 2.0
0 stars 0 forks source link

added loading spinners and a shimmer for load more and userdata #89

Closed eoss-sh closed 1 year ago

eoss-sh commented 1 year ago

Ich habe einen Loading Spinner beim Load More und einen Shimmer für die Userdaten eingebaut. Kannst du mal schauen, ob das vor allem bei den Userdaten so ok ist?

Auf der Index Seite werden die User nun auch kurz mit dem Shimmer angezeigt, da die Session geprüft wird. Hast du eine Idee, wie man das noch verhindern könnte?

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
app-thierry-und-simon ✅ Ready (Inspect) Visit Preview Apr 21, 2023 8:52pm
THRY commented 1 year ago

ok cool. Hab noch im next.config.js die zitadel-domain hinzugefügt, sonst gabs einen error bei mir wegen dem avatar bild.

Noch eine Frage: auf der Index-Seite sollte es keinen Loading state geben bei den Usern, weil die ja Serverseitig geladen werden, nicht?

eoss-sh commented 1 year ago

@THRY ja genau, das ist das komisch… ich glaube, das ist weil die session zuerst abgerufen wird und dies dann den loading state verursacht, ich wollte mal noch veruschen nach den user daten zu prüfen, die sollten ja aus dem fallback vorhanden sein, ich versuche das später noch, ok?