Ticket #406 (closed task: fixed)

Opened 5 months ago

Last modified 4 months ago

Poner opiniones sobre opiniones a la demanda

Reported by: frans Owned by: carlos
Priority: medium Milestone: 1.5 mejoras
Component: front Version: 1.0
Keywords: Cc: juan

Description (last modified by frans) (diff)

por defecto no enseñar las opiniones sobre opiniones, y enseñarlas a la demanda

  • poner "lleva x a favor y x en contra" en lugar de "lleva x [icono+] y x [icono-]" + añadir enlace "ver"
  • cuando desplegado remplazar enlace "ver" por "ocultar"
  • cuando se despliega enseñar las 20 priemras opiniones sobre la opinión + "más" si hay más (ahora creo quer sólo enseñamos 2)
  • sólo cargar texto "Mostrando 1 comentarios de 1" cuando haya más de 20 opiniones

Attachments

retoques opiniones.jpg Download (89.9 KB) - added by juan 5 months ago.
opiniones_y_subopiniones_actual.png Download (53.4 KB) - added by carlos 4 months ago.

Change History

  Changed 5 months ago by frans

  • summary changed from Ponert opiniones sobre opiniones a la demanda to Poner opiniones sobre opiniones a la demanda

Changed 5 months ago by juan

  Changed 5 months ago by juan

  • owner changed from juan to carlos

qué opinais? yo lo veo muucho más claro así. he aprovechado para darle unos martillazos a los vootos y opiniones a nivel de maquetación, son sobre todo de espacios...

ya me comentais

follow-up: ↓ 4   Changed 5 months ago by juan

ah! podríamos aprovechar también para quitar lo de: Mostrando 1 comentarios de 1

Si la paginación no se carga hasta que tenemos 20 opiniones por ej no tenemos por qué mostrar "Mostrando 1 comentarios de 1" puesto que estamos viendo el total a un golpe de vista. Sólo cargaría este texto cuando necesitemos paginar.

in reply to: ↑ 3   Changed 5 months ago by frans

  • description modified (diff)
  • milestone changed from 1.6 propuestas to 1.5 mejoras

Replying to juan:

ah! podríamos aprovechar también para quitar lo de: Mostrando 1 comentarios de 1 Si la paginación no se carga hasta que tenemos 20 opiniones por ej no tenemos por qué mostrar "Mostrando 1 comentarios de 1" puesto que estamos viendo el total a un golpe de vista. Sólo cargaría este texto cuando necesitemos paginar.

me parece mucho mejor :) + Descripción actualizada

  Changed 5 months ago by juan

bien :) seeguimos!

  Changed 5 months ago by frans

  • priority changed from low to medium

  Changed 5 months ago by carlos

  • owner changed from carlos to sergio

Despliegue de subopiniones y nueva línea de resumen de votos recibidos listos. Sergio, ¿te encargas de cambiar a 20 subopiniones por página, y ocultar el "Mostrando ... de ..." así como el botón "más" cuando haya menos de 20?

  Changed 5 months ago by carlos

  • status changed from new to closed
  • resolution set to front ready for revision

  Changed 5 months ago by juan

ojo, que la línea vertical gris acabe justo donde terminan los comentarios, como hemos eliminado el "mostrando de" que no se cargue ese espacio tampoco ;)

  Changed 4 months ago by frans

  • cc juan added

Juan: ¿has revisado ésto? ¿estoy viendo la última versión en dummy?

  • una vez elegido "ver" esta opción debería cambiar a "ocultar"
  • si se muestran todas las opiniones sobre una opinión, quitar "Mostrando 2 comentarios de 2"
  • quitar "(Lleva 2 [mano arriba])" a la derecha de la opinión, que está remplazado por "(Lleva 2 a favor , ver)" debajo del "Opinar sobre esta opinión"
  • después de borrar mi opinión sigue poniendo "Lleva 2..." cuando ya sólo lleva 1
  • quitaría o cambiaría "(de momento no tiene)" cuando no hay opinión sobre la opinión: me parece muy poco explícito

  Changed 4 months ago by juan

aún no lo había revisado. estuve viéndolo hace poco y al ver que faltaban cosas pensé que aún no estaba cerrado. carlos, sergio, es esta la última versión?

follow-up: ↓ 21   Changed 4 months ago by sergio

Falta que saque más de 2 opiniones al desplegar. Cerramos este y abrimos otro en el ml1.6 con solo esto ?

  Changed 4 months ago by sergio

  • status changed from closed to reopened
  • resolution front ready for revision deleted

  Changed 4 months ago by sergio

  • status changed from reopened to closed
  • resolution set to ready for revision

  Changed 4 months ago by juan

  • status changed from closed to reopened
  • resolution ready for revision deleted

falta pegar más el comentario asociado a un vooto, en permalink y en streaming de comentarios. está demasiado separado. ver proto

  Changed 4 months ago by juan

  • owner changed from sergio to carlos
  • status changed from reopened to new

  Changed 4 months ago by carlos

Juan, vamos a aclarar en un mismo ticket el tema de los espacios entre opiniones, subopiniones, etc. Que me hago un lío teniendo estos comentarios desperdigados en varios tickets. Y si a esto le sumamos el baile de márgenes que tenemos que establecer para separar correctamente cualquiera de los casos...

Te adjunto una captura de cómo se pintan ahora mismo. ¿Me listas aquí los cambios que crees oportunos? E intenta darme medidas aproximadas (ej: a la mitad de espacio, el doble de espacio, etc). Gracie mile.

Changed 4 months ago by carlos

  Changed 4 months ago by juan

Hacer cambios en mi opinión/Opinar sobre este comentario + el texto que va por debajo, pegarlo más al comentario del usuario (en el pantallazo que adjuntas más pegado al "estoy de acuerdo"). En realidad creo que lo ideal sería un salto de línea con respecto al comentario del usuario

Las opiniones sobre un vooto están separadísimas. Abre el proto y fusila, creo que así será más fácil, no crees?

  Changed 4 months ago by carlos

  • owner changed from carlos to sergio

Ya he hecho todos los ajustes necesarios en CSS. No obstante, necesito que Sergio modifique el partial _subreviews.php para que haga lo siguiente:

  • Como ya está reflejado en el otro ticket, la etiqueta li class="subreviews-pagination" de la línea 28 no debe aparecer si no hay nada que mostrar ahí.
  • La etiqueta li class="review-new" de la línea 7 tampoco debe aparecer, de lo contrario las subopiniones se separan mucho de la opinión. Una cosa que se puede hacer es establecer la propiedad "display: none" para este li, pero entonces al cargar el formulario se debe hacer un show() sobre ese elemento, o el formulario no aparecerá.

Voy a ver si localizo dónde está el código que carga ese formulario. Si no, Sergio, por favor cámbialo para que tenga el comportamiento descrito.

  Changed 4 months ago by carlos

Sergio, he hecho que review-new esté oculto (display: none), y que cuando se pulse el "Hacer cambios en tu opinión" o el "Opinar sobre este comentario" se haga un show sobre el review-new en cuestión. Pero al pulsar el "Dejarlo como estaba", hay que volver a ocultar el review-new. ¿Le puedes echar un ojo?

in reply to: ↑ 12   Changed 4 months ago by frans

Replying to sergio:

Falta que saque más de 2 opiniones al desplegar. Cerramos este y abrimos otro en el ml1.6 con solo esto ?

ok, sacado para ml 1.7:  http://trac.voota.org/ticket/445

  Changed 4 months ago by sergio

  • status changed from new to closed
  • resolution set to ready for revision

  Changed 4 months ago by frans

  • status changed from closed to reopened
  • resolution ready for revision deleted

  Changed 4 months ago by frans

  • status changed from reopened to closed
  • resolution set to fixed

  Changed 4 months ago by juan

Cuando estoy en el permalink el "hace 4 meses" aparece descolgado...

  Changed 4 months ago by juan

  • status changed from closed to reopened
  • resolution fixed deleted

  Changed 4 months ago by juan

  • owner changed from sergio to carlos
  • status changed from reopened to new

  Changed 4 months ago by sergio

  • status changed from new to closed
  • resolution set to fixed

Juan, acabamos de subir el ml. Voy a cerrarlo, cualquier cosa que veas ponlo en un ticket nuevo. Si ves que son correcciones urgentes que hay que subirla lo pones en critical. ¿ok? thx

Note: See TracTickets for help on using tickets.