Ticket #331 (closed task: fixed)

Opened 6 months ago

Last modified 5 months ago

Reportar error

Reported by: frans Owned by: sergio
Priority: medium Milestone: 1.4 político
Component: front Version: 1.0
Keywords: Cc: juan

Description (last modified by sergio) (diff)

en página político y partido, añadir el bloque "¿Ves algún error?" en la columna derecha, debajo del bloque de enlaces

Juan: adjuntar prototipo

opciones:

  • Faltan datos o son incorrectos
  • Suplantación de identidad
  • Hay algo ofensivo
  • El político está repetido
  • Todo falla, protesto!
  • Todo perfecto, invitar a cañas!

-> enlaza con el contáctanos, con subject pre-completado con opción seleccionada + cuerpo que incluye la url del político o partido en cuestión.

Attachments

select_en_ie7.png Download (11.3 KB) - added by carlos 5 months ago.
comparativa_select.png Download (124.5 KB) - added by carlos 5 months ago.

Change History

  Changed 6 months ago by frans

  • description modified (diff)

  Changed 6 months ago by juan

Ver prototipo en #324: Retoque maquetación opiniones

  Changed 5 months ago by sergio

  • owner changed from sergio to carlos
  • description modified (diff)

  Changed 5 months ago by juan

Ayer comentábamos si sería posible que si se pincha en Todo perfecto, invitar a cañas, llevásemos a la cuenta paypal. Para que no se nos olvide :)

  Changed 5 months ago by sergio

Juan, sí es posible, como no vamos a dejar que inviten a cañas !

Carlos, te lo paso para que lo maquetes, aunque no se si estará hecho ya en alguna parte. Te suena?

  Changed 5 months ago by carlos

No, no estaba. Lo acabo de subir, he creado un partial _report_error.php que es el que se incluye en la página del político y del partido respectivamente (supongo que tendrás que pasar alguna que otra variable al partial para incluir el ID del político o del partido en el formulario).

Sólo me falta buscar una solución Javascript ligera para que IE pinte el select como $DEITY manda.

  Changed 5 months ago by carlos

En IE7 no se puede estilizar más el select (salvo el background) porque éste es pintado usando las propias librerías de Windows (y no el navegador). En IE8 y resto de navegadores se pinta bien. Adjunto captura.

La única solución para dejarlo perfecto pasa por sustituir el select y sus options por una lista ul, y añadir código javascript para simular la selección. La mayoría de los plugins Javascript que he encontrado por ahí están pensados para select de una sola fila (y nosotros pintamos todas las opciones a la vez), así que tendría que basarme en uno de ellos para el reemplazo y luego codificar los estilos.

Yo creo que es mucho curro para una cosa tan nimia, pero vosotros mandáis.

Changed 5 months ago by carlos

  Changed 5 months ago by sergio

Creo que no necesitamos que se vea perfecto. La política general es que no desarrollamos para una versión en concreto de un navegador en concreto.

Si Juan no se opone, tira p'alante.

  Changed 5 months ago by carlos

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

Ok, estoy de acuerdo. Marco entonces como "front ready for revision".

follow-up: ↓ 11   Changed 5 months ago by carlos

Juan, si tienes algo que decir, dilo ahora o calla para siempre ;-)

in reply to: ↑ 10   Changed 5 months ago by juan

Replying to carlos:

Juan, si tienes algo que decir, dilo ahora o calla para siempre ;-)

Por mi parte p'alante. No encuentro necesidad de darle vueltas al tema. Mientras que funcione por mi ok

  Changed 5 months ago by sergio

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

No tenemos subject en el formulario de contacto, así que lo que se precompleta es el mensaje ...

  Changed 5 months ago by sergio

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

  Changed 5 months ago by juan

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

el subject del formulario de contacto es la opción previa que selecciona en el combobox. Subject: invitar a cañas..., todo falla, protesto, etc.

  Changed 5 months ago by juan

  • status changed from reopened to new

  Changed 5 months ago by carlos

  • owner changed from carlos to sergio

  Changed 5 months ago by juan

Ojo a los estilos en la caja de reportar error: - El titular tiene color más oscuro; - La tipo parece más grande; - Se me carga la barra de scroll en varios navegadores, cuando no es necesario...

  Changed 5 months ago by sergio

  • owner changed from sergio to carlos

Carlos, echas un ojo a los últimos comentarios de Juan ?

  Changed 5 months ago by carlos

- El color y tamaño de la tipografía del titular es exactamente igual que el del titular "Enlaces externos": 16px, Georgia negrita.

- Por desgracia, la aparición o no de la barra de scroll no parece que la podemos controlar por CSS. He añadido la propiedad "overflow:hidden;" para que no las muestre, pero no funciona. Como decía más arriba, el control "select" tiene muchísimas limitaciones a la hora de poder estilizarlo a placer (en IE, de hecho, lo pinta una librería del sistema y no el navegador, así que no se puede tocar absolutamente ninguna de sus propiedades mediante CSS en éste browser). Si queremos poder controlar su aspecto al detalle, entonces hay que prescindir de él y elaborar una solución propia mediante una lista no numerada, y elementos seleccionables por javascript (que a su vez marcan el valor en un campo oculto para que el formulario se envíe adecuadamente).

Por cierto, otra cosa que no funciona en ningún navegador salvo Firefox es el padding (margen interior) de las opciones del select. No sé si será mejor quitarlo para que se vea igual en todos.

  Changed 5 months ago by carlos

El color del texto de las opciones del select sí que se estaban pintando más oscuros (negro) que el resto de texto regular de la página (gris oscuro). Lo he cambiado para que se vean en esa tonalidad de gris.

  Changed 5 months ago by sergio

¿Por que tiene que ser un control select?

  Changed 5 months ago by carlos

No tiene por qué serlo :-) Como decía, es el único control HTML que hay más parecido al proto. Pero puede ser una combinación de lista desordenada (UL) con elementos (LI) que contienen las opciones, y Javascript para marcar / desmarcar elementos, rellenar un campo oculto para que se envíe posteriormente la opción seleccionada, etc. Igual que pasa con el ticket #306, mediante Javascript se pueden hacer muchas cosas que no es posible hacerlas usando sólo HTML y CSS (el control del #306 es mucho más complejo, y el proto final creo que se parece bastante a lo que perseguíamos). Por eso decía al principio de debatir sobre este ticket que si nos conformábamos con el renderizado del select (que la verdad sea dicha, no es tan malo), o hacíamos el nuestro propio como hicimos con #306 (que nos llevará más tiempo, pero es perfectamente factible).

  Changed 5 months ago by juan

uy yo aquí ya me he perdido... en realidad hasta podría ser una lista de enlaces, pero es por aquello de que no sea tan fácil pinchar para reportar errores...

  Changed 5 months ago by carlos

Estoy de acuerdo, los enlaces son excesivamente fáciles de pulsar incluso por error. Yo creo que o (1) nos conformamos con el select (¿realmente es tan malo?), o (2) nos curramos un control propio a base de HTML+CSS+Javascript. ¿Qué hacemos?

  Changed 5 months ago by sergio

a favor de conformarnos con el select

  Changed 5 months ago by carlos

Adjunto captura comparativa de cómo se pinta el control en los diferentes navegadores.

Nota: En Chrome / Safari / Firefox, la opción por defecto se colorea de gris como se aprecia en la captura, pero una vez se hace click sobre cualquiera de las opciones, se vuelve azul con en IE.

Changed 5 months ago by carlos

  Changed 5 months ago by juan

Un detalles más: El título de la caja "Algún error?..." y caja que va por debajo no tienen la misma separación que la de "Enlaces externo" con respecto a los enlaces. Me quedo con la distancia del primero, podemos aplicarla a la de los Enlaces externos para que todo quede igual?

  Changed 5 months ago by frans

todo ok, + posible mejora: si usuario login propongo pre-completar su nombre (nombre, no necesario incluir apellidos) y además si tenemos su email prec-completar su dirección email

  Changed 5 months ago by carlos

  • owner changed from carlos to sergio

Separación de título - contenido en enlaces externos arreglada. Sergio, te paso el ticket para el último punto comentado por frans.

  Changed 5 months ago by sergio

  • status changed from new to assigned

Pongo lo de precompletar para el siguiente ml ( #411 )

  Changed 5 months ago by sergio

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

  Changed 5 months ago by sergio

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

  Changed 5 months ago by sergio

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

  Changed 5 months ago by frans

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

  Changed 5 months ago by frans

  • status changed from reopened to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.