Ticket #277 (closed task: fixed)

Opened 7 months ago

Last modified 4 months ago

Añadir permalink a opiniones

Reported by: frans Owned by: sergio
Priority: low Milestone: 1.5 mejoras
Component: front Version: 1.0
Keywords: Cc: juan, sergio

Description (last modified by frans) (diff)

en "hace x días" permalink a opinión

  • falta prototipo página opinión, incluyendo comentarios sobre la opinión
  • url = voota.es/r/[id_review] (a confirmar por Sergio)
  • corregir enlace desde página de usuario, pública y privada (ahora el ancla "Ir a comentario" no funciona): enlazar con permalink de la opinión raíz
  • poner permalink para opinión y para opinión sobre opinión

Attachments

Voota_Permalinks.3.png Download (326.0 KB) - added by juan 5 months ago.
Voota_Permalinks_4.png Download (332.5 KB) - added by juan 5 months ago.
Pantallazo.png Download (63.1 KB) - added by sergio 4 months ago.

Change History

  Changed 5 months ago by frans

  • cc sergio added
  • description modified (diff)
  • milestone changed from 2 to 1.5 mejoras

  Changed 5 months ago by juan

Un pequeño matiz: hacemos permalinks de los textos de los vootos, pero igual también es importante tener permalinks de las opiniones sobre un Vooto, no?

Changed 5 months ago by juan

  Changed 5 months ago by juan

Otra cosa importante: si hay muchos comentarios cargamos el botón "más" de paginación.

  Changed 5 months ago by sergio

  • owner changed from juan to carlos

  Changed 5 months ago by juan

  • owner changed from carlos to sergio

  Changed 5 months ago by sergio

  • owner changed from sergio to carlos

  Changed 5 months ago by frans

  • description modified (diff)

Juan, en la página de la opinión, falta una mención a la "entidad" sobre la que se ha publicado esta opinión (político, partido, propuesta u otra opinión)

  Changed 5 months ago by juan

Actualizado .png haciendo mención a la "entidad" sobre la que se ha publicado esta opinión (político, partido, propuesta u otra opinión). Sorry Carlos!

Changed 5 months ago by juan

  Changed 5 months ago by carlos

No worries.

Lo de la "entidad" os ha calado hondo, ¿eh? :)

  Changed 5 months ago by carlos

Casi lo tengo. Sergio, en la nueva página de la opinión no se pintan ni el enlace para añadir subreview ni las subreviews, porque $reviewable no se evalúa a true. ¿Le echas un ojo mientras lo finiquito? Gracias.

  Changed 5 months ago by carlos

90% listo, a falta de poder ver cómo se pintan el enlace para opinar y las subopiniones una vez Sergio pueda echar un ojo a lo de $reviewable.

  Changed 5 months ago by carlos

Sergio, en showSuccess.php hay que sustituir el nombre del político/partido/propuesta en el enlace, así como su URL.

  Changed 5 months ago by carlos

  • owner changed from carlos to sergio

  Changed 5 months ago by juan

Ojo, el "Hace..." se carga en diferentes alturas en función de donde estemos: Si estamos en el streaming de comentarios queda muy abajo, mientras que en el permalink ok. Dejaría ambos a la altura de este último.

  Changed 5 months ago by carlos

¡Estás en todo, Juan! :-) Tienes toda la razón, muy bien visto. He pegado más la fecha en los comentarios de la lista.

Por cierto, en el streaming de comentarios de un usuario (página "Tus comentarios y vootos"), el "Hace..." también debería ser un permalink a la página del comentario, ¿no?

  Changed 5 months ago by juan

Toootalmente de acuerdo. Lo incluímos también en esa sección. No hace falta crear ticket, lo ha hablado con sergi

  Changed 5 months ago by carlos

Ok. Hecho.

  Changed 5 months ago by juan

  • owner sergio deleted

Ojo. Cuando un usuario ha votado a favor o en contra, pero no ha comentado nada, tanto de una opinión como de un político/partido/propuesta, el permalink debería indicar un texto diciendo que no ha comentado nada.

En el permalink del Vooto y comentario propongo, si no tiene texto colocar simplemente: Sin comentario.

  Changed 5 months ago by juan

Perdón, mejor así: Este vooto no tiene comentario aún. Esta opinión no tiene comentario aún.

  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

correcciones en  http://dummy.voota.es/r/1546 (puede que muchas de las siguientes correcciones correspondan al ticket de ocultar las opiniones sobre opiniones):

  • si elijo "ver" opiniones sobre la opinión, falla la numeración (1 a 4 en lugar de 1 a 2)
  • 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"
  • si edito mi opinión sobre la opinión, después de guardarla debería dejarme con las opiniones sobre la opinión visibles (no ocultarlas sin que lo haya pedido)
  • si ya he opiniado sobre esta opinión cambiar texto "Opinar sobre este comentario" por "Hacer cambios en tu opinión"
  • falta espacio entre primera opinión sobre la opinión, y la segunda
  • 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

  Changed 4 months ago by frans

  • cc juan added
  • owner set to sergio
  • status changed from reopened to new

  Changed 4 months ago by juan

avatar del usuario y el nombre de usuario se alinean de forma aleatoria, unas veces más alto, otras más bajo. tienen que estar alineados como en el resto del web.

tamaño de fuente en las opiniones dentro del permalink: tiene que ser igual al resto del web también, los nombres de usuario no llevan fuente más grande.

eliminar el texto "mostrando 2 de 2". esto sólo lo enseñamos cuando hay más de 20 comentarios.

hay que darle un repaso importante a esto

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

  • owner changed from sergio to carlos

Me falta esto

  • si edito mi opinión sobre la opinión, después de guardarla debería dejarme con las opiniones sobre la opinión visibles (no ocultarlas sin que lo haya pedido)

¿Abrimos ticket en otra ml con ello?

Carlos, miras esto

  • falta espacio entre primera opinión sobre la opinión, y la segunda

y los ajustes que propone Juan

A nadie le salen los numeritos del ol ? Yo lo veo un poco raro

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

"numeritos del ol"? no entender...

in reply to: ↑ 25   Changed 4 months ago by sergio

Replying to juan:

"numeritos del ol"? no entender...

Adjuntando pantallazo (veanse el 1. 2. 3.)

Changed 4 months ago by sergio

  Changed 4 months ago by juan

Sí, yo también lo veo. Creo que esto aún no está maquetado...

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

Replying to sergio:

Me falta esto * si edito mi opinión sobre la opinión, después de guardarla debería dejarme con las opiniones sobre la opinión visibles (no ocultarlas sin que lo haya pedido) ¿Abrimos ticket en otra ml con ello?

ok, apuntado en  http://trac.voota.org/ticket/444 para ml siguiente

follow-up: ↓ 31   Changed 4 months ago by carlos

Lo de los numeritos en el ol: Sí que estaba todo maquetado, aunque en la versión del master faltaban dos propiedades CSS en sf_review.css. Puede que se perdieran en el tránsito de subversion a git, no sé. Pero el caso es que ya está arreglado.

En cuanto a las otras dos cosillas:

"avatar del usuario y el nombre de usuario se alinean de forma aleatoria, unas veces más alto, otras más bajo. tienen que estar alineados como en el resto del web".

Esto no lo entiendo, o quizás es que no lo puedo reproducir. ¿Puedes mandar un pantallazo, Juan?

"tamaño de fuente en las opiniones dentro del permalink: tiene que ser igual al resto del web también, los nombres de usuario no llevan fuente más grande."

Sólo las de opiniones dentro del permalink, ¿verdad? El nombre de usuario de la opinión a la que pertenece sí que va más grande, como en el proto, ¿cierto?

Sergio, una cosa: Para que la línea izquierda no crezca tanto, el li con clase "subreviews-pagination" no debería pintarse si no hay paginación que mostrar.

  Changed 4 months ago by carlos

He puesto los nombres de usuarios en las subopiniones a 13px. El de la opinión principal (la del permalink) lo he dejado a 16px. Ya me dices algo, Juan.

in reply to: ↑ 29 ; follow-up: ↓ 32   Changed 4 months ago by juan

Replying to carlos:

Lo de los numeritos en el ol: Sí que estaba todo maquetado, aunque en la versión del master faltaban dos propiedades CSS en sf_review.css. Puede que se perdieran en el tránsito de subversion a git, no sé. Pero el caso es que ya está arreglado. En cuanto a las otras dos cosillas: "avatar del usuario y el nombre de usuario se alinean de forma aleatoria, unas veces más alto, otras más bajo. tienen que estar alineados como en el resto del web". Esto no lo entiendo, o quizás es que no lo puedo reproducir. ¿Puedes mandar un pantallazo, Juan?

Me refiero al avatar, al nombre de usuario y al "hace x" y es porque hemos aumentado la fuente en las opiniones sobre un vooto en los permalinks. Fíjate bien el proto, sólo aumentamos de fuente en el permalink de usuario. Las opiniones mantienen el mismo formato.

"tamaño de fuente en las opiniones dentro del permalink: tiene que ser igual al resto del web también, los nombres de usuario no llevan fuente más grande." Sólo las de opiniones dentro del permalink, ¿verdad? El nombre de usuario de la opinión a la que pertenece sí que va más grande, como en el proto, ¿cierto?

No, esto igual que en el resto del web. En el proto sólo tiene fuente más grande el permalink, lo demás permance igual.

Sergio, una cosa: Para que la línea izquierda no crezca tanto, el li con clase "subreviews-pagination" no debería pintarse si no hay paginación que mostrar.

in reply to: ↑ 31   Changed 4 months ago by carlos

Replying to juan:

Me refiero al avatar, al nombre de usuario y al "hace x" y es porque hemos aumentado la fuente en las opiniones sobre un vooto en los permalinks. Fíjate bien el proto, sólo aumentamos de fuente en el permalink de usuario. Las opiniones mantienen el mismo formato.

Vale, eso ya está arreglado, ahora sólo se pinta la fuente más grande para la opinión a la que pertenece el permalink. Aunque veo que en dummy aún no está.

Sólo las de opiniones dentro del permalink, ¿verdad? El nombre de usuario de la opinión a la que pertenece sí que va más grande, como en el proto, ¿cierto?

No, esto igual que en el resto del web. En el proto sólo tiene fuente más grande el permalink, lo demás permance igual.

Sí, eso es exactamente lo que te decía: La fuente más grande sólo en el autor de la opinión que pertenece al enlace permanente (permalink). Las otras opiniones sobre esta opinión llevan el nombre del autor con la tipografía de 13px habitual.

Sergio, una cosa: Para que la línea izquierda no crezca tanto, el li con clase "subreviews-pagination" no debería pintarse si no hay paginación que mostrar.

Sobre esto: Sergio, para que tengas más detalles: el elemento li class="subreviews-pagination" no debe mostrarse cuando no se muestra tampoco el 'Mostrando...' ni el botón 'más'. Tienes puesto ahí varias condiciones, pero la condición que rodea al li es if($total > 0), y ésta pinta el li cuando no hay paginador que mostrar.

  Changed 4 months ago by carlos

  • owner changed from carlos to sergio

  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
Note: See TracTickets for help on using tickets.