Menu

#176 Add some Warning message

v4.3
closed
nobody
None
5
2016-04-14
2010-11-03
No

Agregar tambien mensages de advertencias o warning (de color amarillo), por que hay ocaciones en que se quiere advertir al usuario de algo pero no es un error ni un mensage de alguna operacion satisfactoria. entonces pienso que deberian agregar alguna especie de mensages de ese tipo o darle la opcion de cambiarle el color al addMessage().

Discussion

  • Javier Paniza

    Javier Paniza - 2010-11-04

    Hola Pedro,

    tu sugerencia me parece muy útil.
    ¿Estarías dispuesto a implementarlo tú? Yo te puedo guiar.

     
  • Anonymous

    Anonymous - 2010-11-04

    Si con mucho gusto

     
  • Anonymous

    Anonymous - 2010-11-04

    Hola Javier,

    pienso que agregandole una sobre carga al addMessage seria una buena opcion, para que se pueda setear el color de los mensages o elegir algun Enun MessageType {xxx,xxx,xx}.
    y asi aunque se maneje como un Message se le va a presentar al usuario de otro modo que le prestara mas atencion que a los mensages verdes.

    o
    crear un nuevo metodo identico a addMessages(). pero que se llame AddWarning()

    que dices?

     
  • Javier Paniza

    Javier Paniza - 2010-11-05

    > crear un nuevo metodo identico a addMessages(). pero que se llame
    > AddWarning()
    Esto me gusta más, sencillamente por que es más coherente con lo que ya hay. Ahora tenemos addMessage() y addError() como métodos separados.
    Mira como está implementado addMessage() y haz exactamente lo mismo para addWarning(), y por supuesto preguntame cualquier duda.

     
  • Anonymous

    Anonymous - 2010-11-08

    Hola Javier,

    ya he puesto ha funcionar el addWarnings(), pero sale el mensage sin la decoracion debida, osea sale el mensage con el color de letra que le asigne en openxava.css pero para aregarle el cuadro y la imagen igual que addMessages tienes que darme una mano o indicarme donde es que se asigna esa decoracion por que no la encontre por parte.

    ahh y por favor me indicas como te puedo enviar los cambios que he hecho para que lo revises!

    Gracias,
    Pedro Garcia

     
  • Javier Paniza

    Javier Paniza - 2010-11-09

    Hola Pedro,

    has de añadir un método getWarnings() en la clase Style, JetSpeed2Style, WebSphereStyle, Liferay43 y Liferary41Style.
    En Style pon "warnings" como valor, y en los demas usa "portlet-msg-alert" que es el estilo estándar de los portales para mensajes de advertencia. En el html que generas para el mensaje obten el estilo de la clase Style.

    >por favor me indicas como te puedo enviar los cambios que he hecho
    >para que lo revises!
    Comprime todas las clases, jsp, css, etc. que has tocado en un zip y adjuntalo en esta incidencia. Yo lo revisaré, si veo pegas te las compentaré para que lo modifiques, y si está bien lo integraré para que esté disponible en la próxima versión de OX

     
  • Anonymous

    Anonymous - 2010-11-09

    Archivos modificados para agregar el metodo addWarning

     
  • Anonymous

    Anonymous - 2010-11-09

    Hola javier,

    He subido los archivos que he modificado. probe el addWarning() en una accion y funciono bien pero en otra accion me daba un null. tendre que debuguearlo para verificar por que sera.

    Pedro Garcia.

     
  • Javier Paniza

    Javier Paniza - 2010-11-10

    Hola Pedro,

    > me daba un null. tendre que debuguearlo para verificar
    Esperaré a que corrijas ese problema antes de incluir tu código en OX.

     
  • Javier Paniza

    Javier Paniza - 2016-04-14
    • status: open --> closed
    • Group: --> v4.3
     

Log in to post a comment.