mvk-team42 / Veracitor

An application that studies the trust in a network of users, sources and articles
6 stars 2 forks source link

Web page loading state #56

Open johnbrynte opened 11 years ago

johnbrynte commented 11 years ago

När sidan laddas görs en request till servern för att hämta information om användaren och alla taggar. Detta gör att sidan kan se konstig ut när den laddas eller i värsta fall inte ladda klart om det blir fel i requesten, något som händer oroande ofta. Även olika script laddas inte alltid in av någon anledning utan kräver att sidan laddas om för att försöka på nytt.

Jag föreslår att vi skapar ett loading state som sidan går in i när den laddas då alla nödvändiga requests hanteras. Under tiden kan en liten loading-popup eller liknande visas för att ge användaren feedback.

antonaut commented 11 years ago

Gjorde så att loading visas by-default i index tills dess att $('document').ready() körs. se commit https://github.com/mvk-team42/Veracitor/commit/40be0ca7f248a416be691cb0f534bda2b6791584

johnbrynte commented 11 years ago

Nja, document.ready använder vi redan i vera.js. Problemet jag pratar om handlar om initiala requests som misslyckas vilket gör att sidan inte laddas. Loading-statet behöver alltså en blockande funktion som gör alla nödvändiga requests innan sidan laddas.

antonaut commented 11 years ago

Fattar inte riktigt vad jag tänkte med när jag läste ditt förslag... Tänkte att du menade att sidan inte visades snyggt förrän allting laddats in... >.<

Vi skulle antagligen kunna lösa inladdningen genom att ha en script-loader/modulhanterare (typ require.js löser detta) Kan göra en testbranch för det? :)

Vad gäller requests:en behövs väl bara att .fail() implementeras med vettiga defaultvärden så länge på de requests som failar oftast.

johnbrynte commented 11 years ago

Require låter faktiskt najs! Min tanke om requesten var att de ligger o tuggar medan laddsidan visas, sen när de är klara kallas en metod, eller sätts en variabel, som berättar att sidan är färdigladdad!

antonaut commented 11 years ago

Eller så sker ett eget ready-event?