emergenzeHack / terremotocentro

Il sito del progetto TerremotoCentroItalia
http://terremotocentroitalia.info
MIT License
66 stars 11 forks source link

Ritoccare ricerca e inserire loading sul sito #414

Closed ghost closed 8 years ago

ghost commented 8 years ago
iltempe commented 8 years ago

@tailot possiamo verificare per favore? mi pare che alla prima ricerca il loader non compaia e compare invece solo dopo un refresh della pagina searchissues. grazie

ghost commented 8 years ago

Si, attualmente quel search è bloccante per cui sostanzialmente si paralizza tutto, bisognerebbe renderla più efficiente

olistik commented 8 years ago

@tailot Ottima soluzione :-) in https://github.com/emergenzeHack/terremotocentro/commit/394fbfda4aed459b78451ad97e3be2353bcdbab3 avevo provato un approccio diverso ma lo spinner restava bloccato dal processo della request e e di rendering.

Il fatto di iniettare il codice JS direttamente in page tende a creare qualche problemino in fase di sviluppo: il watch del server è lento a rigenerare la pagina di search..

Se per voi ha senso posso provare a riscrivere la pagina in modo che si appoggi ad un json generato staticamente ad ogni build del progetto (e quindi esterno alla pagina), in modo che la l'html della search resti sempre snello.

ghost commented 8 years ago

@olistik era una cosa che mi martellava nella testa. Si sembra una buona idea staticizzare il json.

iltempe commented 8 years ago

il json se intendete quello delle segnalazioni c'è già. https://github.com/emergenzeHack/terremotocentro/blob/master/_data/issuesjson.json @olistik @tailot

iltempe commented 8 years ago

@tailot una volta che il search è di vostro gradimento e lo avete abbellito io lo piazzerei in prima pagina
togliendo tutto tranne i pulsantoni. Togliendo anche la lista delle segnalazioni. Stile Google. Che dite? untitled

iltempe commented 8 years ago

Ho tirato giù un mockup di una possibile home page. che dite? si può lavorare in questa direzione? ovviamente va messo a posto CSS, validata etc... http://terremotocentroitalia.info/index2/ fatemi sapere