search

hhba / mapa76

Plataforma de investigación periodística
http://analice.me/
31 stars 8 forks source link

Vista de Documento: Mejoras y arreglos #187

Closed pjnovas closed 10 years ago

pjnovas commented 10 years ago
pjnovas commented 10 years ago

@marianet @malev

Las entidades no son clickeables

Arme el modal para las mentions ... pero el server retorna mal los IDs para los reemplazos en el texto. En teoria una vez que el server retorne lo que corresponde, debería funcionar.

  • El paginador no hace nada (cambia el numerito pero no pasa de página).
  • en un doc con mucho texto, da la impresión de hacer muchas queries cada vez que se hace scrolling down.
  • El scroll down no anda

Estuve probando en mi local conectado a Staging y funciona bien, esto es codigo javascript que corre en el cliente, no debería cambiar entre entornos si es el mismo compilado. Lo que puede ser es el diseño de como están las páginas, al tener todas un alto distinto (dependiendo de la cantidad de texto), creo yo, siempre va a parecer extraña la forma en que trabaja el paginado (a nivel usabilidad), por ahí poniendo un alto fijo para cada pagina?, creo que se va a sentir mas natural.

[Mariano] Tal vez el problema es que no anda el scrolling y no se ven todas las páginas de los documentos. Se ven algunas.

No hay punto y aparte en los párrafos.

El cliente no tiene la información para saber cuando cerrar un párrafo. Es necesario que el servidor le diga de alguna forma, creo que la mas fácil (para el cliente) es que meta un caracter en el punto de corte ... no se como va a implicar este cambio en el reemplazo de entidades.

Falta la presentación de resultados

Hay alguna url para el Search dentro de un doc?, esto no es algo simple de armar, no creo que lo pueda tener para este viernes.

[Mariano] Para mí debería ser el search igual al que hay en la lista de documentos pero sólo sobre ESE documento.

Las búsquedas debíeran tener url unico

El template de diseño está roto, no está respetando la propuesta gráfica original.

La forma en que está armada la maqueta de HTML (por ejemplo: el titulo del doc en la primer página) lo veo bastante complejo de implementar. Y, en mi opinion, al moverse de página o ingresar en otra que no sea la primera se pierde cual es el titulo del documento. Que quieren hacer con esto?

No hay spinner indicando que las cosas se están cargando

Cual es el spinner que debería usar?

Mucho texto jaja ... si tengo las respuestas para mañana tipo 14hs puede que cierre algunas a la tarde (sin presión, pero con presión :dancers: ) Saludos!

malev commented 10 years ago

Arme el modal para las mentions ... pero el server retorna mal los IDs para los reemplazos en el texto. En teoria una vez que el server retorne lo que corresponde, debería funcionar.

Podrías ampliar un poco más esto? Qué URL retorna mal los IDs?

El "pager" para paginado debería funcionar, le puse mucha onda a eso. Lo mismo al scroller. El 2do tiene una falla cuando haces scrolling muy rápido.

El cliente si sabe cuándo termina un párrafor: https://github.com/hhba/mapa76/blob/staging/aeolus/app/views/Document/Page.js#L62

No, no hay URL de search en vista de documentos. Pero igual podrías trabajar en una vista de resultados. con un JSON inventado. Después hago que el server devuelva eso y listo. Pensa que una búsqueda en la vista de documentos te debería permitir navegar por los resultados a través de todo el documento.

HTML: Coincido con vos, por eos lo hice así. Ahora... eso es de verdad un gran problema?

Qué spineer? el que vos quieras siempre y cuando no sea gigante.

pjnovas commented 10 years ago

@malev

Arme el modal para las mentions ... pero el server retorna mal los IDs para los reemplazos en el texto. En teoria una vez que el server retorne lo que corresponde, debería funcionar.

Podrías ampliar un poco más esto? Qué URL retorna mal los IDs?

La request de /pages donde retorna las entidades para reemplazar en la página, el id que da para una entidad ... por ejemplo { text: "Jose Perez", id: 12345678 } cuando pido esa persona me retorna un 404 y si me fijo en el menú derecho de people, la persona "Jose Perez" tiene otro id, por lo que entiendo porque retorna un 404. Habría que ver si eso es porque el doc es viejo y la data está rota porque las entidades cambiaron ...

Para probarlo, simplemente abrí un doc y dale click a una entidad .. mirando la consola en network para ver la request que hace. (usa el atributo eid del span)

No, no hay URL de search en vista de documentos. Pero igual podrías trabajar en una vista de resultados. con un JSON inventado. Después hago que el server devuelva eso y listo. Pensa que una búsqueda en la vista de documentos te debería permitir navegar por los resultados a través de todo el documento

Este punto queda afuera entonces, no voy a llegar a cerrarlo.

HTML: Coincido con vos, por eos lo hice así. Ahora... eso es de verdad un gran problema? Para mi no es un problema, los bugs los levantó @marianet jeje ... que opinas Mariano?

Qué spineer? el que vos quieras siempre y cuando no sea gigante. Oka, me busco uno y lo mando hoy

marianet commented 10 years ago

Detecta salto de párrafo, pero hay algunos punto a partes que no los detecta, por ende, no hace el punto aparte. Hay varios textos que subí que son diálogos, y no está respetado el formato original. Tendría que tener en cuenta supongo que los BR, o algo así.

Con respecto al html, lo discutimos por chat con pablo primero.

2014/1/16 Marcos Vanetta notifications@github.com

Arme el modal para las mentions ... pero el server retorna mal los IDs para los reemplazos en el texto. En teoria una vez que el server retorne lo que corresponde, debería funcionar.

Podrías ampliar un poco más esto? Qué URL retorna mal los IDs?

El "pager" para paginado debería funcionar, le puse mucha onda a eso. Lo mismo al scroller. El 2do tiene una falla cuando haces scrolling muy rápido.

El cliente si sabe cuándo termina un párrafor:

https://github.com/hhba/mapa76/blob/staging/aeolus/app/views/Document/Page.js#L62

No, no hay URL de search en vista de documentos. Pero igual podrías trabajar en una vista de resultados. con un JSON inventado. Después hago que el server devuelva eso y listo. Pensa que una búsqueda en la vista de documentos te debería permitir navegar por los resultados a través de todo el documento.

HTML: Coincido con vos, por eos lo hice así. Ahora... eso es de verdad un gran problema?

Qué spineer? el que vos quieras siempre y cuando no sea gigante.

— Reply to this email directly or view it on GitHubhttps://github.com/hhba/mapa76/issues/187#issuecomment-32479347 .

Mariano Blejman - Knight International Journalism Fellow International Center For Journalists mblejman@knight.icfj.org http://www.icfj.org/our-work/argentina-launch-innovation-challenge-promote-development-digital-media-latin-america Hacks/Hackers Buenos Aires marianet@he.net mob: +5491150080629 skype: jfk_knoppixxx twitter: @blejman @supleno @cult_digital @hackshackersba LinkedIn: http://linkd.in/97kuH9

pjnovas commented 10 years ago

@malev @marianet me fije en el documento Yorio, Rodolfo Daniel (2011.02.18) Sin corregir.txt entre otros para el tema de los parrafos, el server no está devolviendo los \n dentro del texto, por lo que habría que ver eso y en teoría volvería a funcionar.

marianet commented 10 years ago

Alguien vio cómo están las entidades en este documento? http://analiceme.urdamales.com.ar/documents/527cefd83ea2af0e79000015/#3

2014/1/16 Pablo Novas notifications@github.com

@malev https://github.com/malev @marianet https://github.com/marianetme fije en el documento Yorio, Rodolfo Daniel (2011.02.18) Sin corregir.txt entre otros para el tema de los parrafos, el server no está devolviendo los \n dentro del texto, por lo que habría que ver eso y en teoría volvería a funcionar.

— Reply to this email directly or view it on GitHubhttps://github.com/hhba/mapa76/issues/187#issuecomment-32544361 .

Mariano Blejman - Knight International Journalism Fellow International Center For Journalists mblejman@knight.icfj.org http://www.icfj.org/our-work/argentina-launch-innovation-challenge-promote-development-digital-media-latin-america Hacks/Hackers Buenos Aires marianet@he.net mob: +5491150080629 skype: jfk_knoppixxx twitter: @blejman @supleno @cult_digital @hackshackersba LinkedIn: http://linkd.in/97kuH9

pjnovas commented 10 years ago

para el punto:

no se puede clickear en los checks de los resultados de busqueda y tambien despues de "volver"

Otra vez ... el plugin pedorro iCheck ... le di unas cuantas vueltas pero se rompe cuando se re-bindea un par de veces (tambien probé destruirlo antes) ... en mi opinión, ese plugin no esta preparado para laburar con vistas dinámicas como (ej. un MVC como Backbone).

Habría que buscar otro plugin o bien usar solo checkbox nativos.

malev commented 10 years ago

checkboxes nativos!!!!

malev commented 10 years ago

La request de /pages donde retorna las entidades para reemplazar en la página, el id que da para una entidad ... por ejemplo { text: "Jose Perez", id: 12345678 } cuando pido esa persona me retorna un 404 y si me fijo en el menú derecho de people, la persona "Jose Perez" tiene otro id, por lo que entiendo porque retorna un 404. Habría que ver si eso es porque el doc es viejo y la data está rota porque las entidades cambiaron ...

Fixed, ahora deberías tener un attribute: entity_id esa es la id que tenes que usar :)

pjnovas commented 10 years ago

@malev los campos entity_id me vienen todos en null para http://analiceme.urdamales.com.ar/api/v2/documents/52bc79e63ea2af92b0000012/pages

pjnovas commented 10 years ago

@malev Ahi mande el search de un doc a otro branch fullDocSearch, no hay maqueta para el resultado de un solo doc y se me complica bastante armar eso con los CSS en el SASS de aphrodite ... en teoria ahi esta armada la busqueda con un doc resultado simulado de testeo.

Cuando este la maqueta y el CSS habria que hacer un pull de ese branch ... testear, tunear y mergear.

En teoria de programacion no le falta nada (por ahi el boton de "volver al documento" que tampoco está en la maqueta)

pjnovas commented 10 years ago

@malev @marianet Agregué el Spinner:

Es programatico, no imagen, se puede personalizar desde las settings (aca se pueden cambiar).

Marcos, agregue una biblioteca mas, asi que tenes que correr grunt stage y pasar el vendor.js de nuevo. El plugin que agregue del spinner es este

pjnovas commented 10 years ago

Ah ... como un detalle no menor ... pueden entrar a la pagina del plugin, modificar ahi el que quieren y cuando terminan reemplazan el json en las settings y lo tienen en la app :)

Ojo con las props top left y color