Skip to content
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

Infinite scroll vero #57

Closed
paolomainardi opened this issue Jan 29, 2013 · 18 comments
Closed

Infinite scroll vero #57

paolomainardi opened this issue Jan 29, 2013 · 18 comments
Assignees

Comments

@paolomainardi
Copy link
Member

Visto che il dataset arriverà a quasi 1000 candidati e che la pagina è già al momento di una pesantezza unica, direi che è priorità massima implementare un infinite scroll vero (oppure un pager) altrimenti diventa inutilizzabile.

Me la assegno io, se qualcuno vuole lavorarci mi faccia sapere, altrimenti parto con un'implementazione il prima possibile.

@ghost ghost assigned paolomainardi Jan 29, 2013
This was referenced Jan 30, 2013
@paolomainardi
Copy link
Member Author

@stefanoduri Questo è il codice: https://github.com/spaghetti-open-data/twitAntonio/blob/master/controllers/mainController.js#L20

In realtà basterebbe creare i link del paginatore, anche se ho sempre pensato che la cosa migliore su quella pagina è usare una specie di infinte scroll che carica mentre si scrolla.

@gaspa
Copy link

gaspa commented Feb 3, 2013

@paolomainardi rispondo qui, che è il posto giusto: ci sta lavorando @stefanoduri ? @stefanoduri eventualmente: come sei messo e come posso aiutarti?

@stefanoduri
Copy link
Contributor

no, non credo sia il caso che io metta le mani sul codice, ho capito che dovrei studiarmi troppa roba e non ho tempo. Count me out

@gaspa
Copy link

gaspa commented Feb 3, 2013

@stefanoduri ack, come vuoi. (sicuro? :D :D :D )
@paolomainardi: avevi già guardato qualcosa?

@paolomainardi
Copy link
Member Author

Ste, peccato :/ nella parte di analytics però ci servi ;)

Andrea, ancora no, se vuoi questa issue è tutta tua :)
Il giorno 03/feb/2013 21:46, "Andrea Gasparini" notifications@github.com
ha scritto:

@stefanoduri https://github.com/stefanoduri ack, come vuoi. (sicuro? :D
:D :D )
@paolomainardi https://github.com/paolomainardi: avevi già guardato
qualcosa?


Reply to this email directly or view it on GitHubhttps://github.com//issues/57#issuecomment-13053929.

@stefanoduri
Copy link
Contributor

mi sto anche chiedendo: ma chi è che vuole scrollare o sfogliare centinaia di nomi? forse avrebbe più senso presentare casualmente (meglio: secondo un criterio basato ad es. sul numero di tweet: "I 10 più attivi", oppure: "gli hashtag più utilizzati sono A, B, C: ecco chi ne parla", roba così) un numero fisso di candidati sulla home e rimandare alle funzioni di ricerca per gli altri

@gaspa
Copy link

gaspa commented Feb 4, 2013

Ci sono quasi, direi che possiamo usare questo:
http://www.infinite-scroll.com/infinite-scroll-jquery-plugin/
( ho già un affare che sfunzionicchia, devo solo giocare un po' con gli URL :) )

@paolomainardi
Copy link
Member Author

Grandissimo @gaspa! Questo è un upgrade della piattaforma, fondamentale!

@gaspa
Copy link

gaspa commented Feb 4, 2013

@paolomainardi ho una versione che funziona sulle pagine ordinate (cioè, se hai usato il filtro di ricerca), altrimenti secondo me fa casino. (ma forse neanche tanto)
Se vuoi darci un'occhiata e mi dici che ne pensi:
https://github.com/gaspa/twitAntonio/tree/infinitescrolling

@paolomainardi
Copy link
Member Author

@gaspa Grande!

Adesso non ho tempo di dargli un occhio, questa sera ti aggiorno! 👍

@gaspa
Copy link

gaspa commented Feb 4, 2013

@paolomainardi ora mi sembra a posto anche la visualizzazione "no search".

Giusto due robe:

  • è sparito il checkbox per l'ordine della ricerca (asc/desc, per intenderci). e' voluto?
  • il testo: "...sono presenti gli account twitter di 15 candidati..."... o si cambia il testo, oppure mettiamo il numero fatto con una query a parte.

@paolomainardi
Copy link
Member Author

@gaspa

  1. Si, ho preferito impostare dei valori di default, se vedi lo switch/case, giusto per semplificare un po l'interfaccia.
  2. Intendi la stampa del numero dei risultati ?

Mi aggiungi su gmail ?

@gaspa
Copy link

gaspa commented Feb 4, 2013

  1. k, giusto per esserne certo
  2. si'.

non ho il tuo contatto, te l'ho scritto su facciabuco... [mix dei social network,olè.. :D ]

@nelsonmau
Copy link
Member

@gaspa il testo di mep.legth andrebbe aggiustato sono d'accordo ma era legato anche alla nuova toolbar #75

@gaspa
Copy link

gaspa commented Feb 4, 2013

@nelsonmau ok, se ne parla di là ;)

@nelsonmau
Copy link
Member

ok vengo se mi volete ;) ma dove devo venì?Il 04/02/2013 23:10, Andrea Gasparini
  ha scritto:

  @nelsonmau ok, se ne parla di là ;)

    —
    Reply to this email directly or view
      it on GitHub. 

-- 

Andrea Nelson Mauro - Datajournalist

http://www.dataninja.it
http://twitter.com/nelsonmau

@gaspa
Copy link

gaspa commented Feb 4, 2013

chiudoooooooo \o/

@gaspa gaspa closed this as completed Feb 4, 2013
@paolomainardi paolomainardi reopened this Feb 7, 2013
@paolomainardi
Copy link
Member Author

Pare ci siano problemi di duplicazione dei risultati, @gaspa poi fammi sapere come sei andato avanti e come possiamo aiutarti a chiudere questa issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants