[Liverpool Template] Regarding feedback messages

[Liverpool Template] Regarding feedback messages

  
Forge Component
(14)
Published on 11 Nov (4 weeks ago) by OutSystems Labs
14 votes
Published on 11 Nov (4 weeks ago) by OutSystems Labs
Hi,

I am wondering why the liverpool decided to "hide" the :before in the feedbackmessages.
furthermore, the css on that is somewhat strange and a little bit redundant.

.Feedback_Message_Error:before,
.Feedback_Message_Warning:before,
.Feedback_Message_Success:before,
.Feedback_Message_Info:before {
    display: none;
}

.Feedback_Message_Error:before {
    content: "?";
}

.Feedback_Message_Success:before {
    content: "\f05d";
}

.Feedback_Message_Info:before {
    content: "\f05a";
}

.Feedback_Message_Warning:before {
    content: "\f06a";
}

.Feedback_Message_Success {
    background-color: #ACC965;
    color: #333;
}

.Feedback_Message_Warning {
    background-color: #F2C94E;   
    color: #333;
}

.Feedback_Message_Info {
    background-color: #AABDCF;
    color: #333;
}

/* Feedback messages */
.Feedback_Message_Error:before,
.Feedback_Message_Warning:before,
.Feedback_Message_Success:before,
.Feedback_Message_Info:before {
    display: none;
}

Hello Statler,

Thanks for your feedback. We will take it in consideration for future releases.
There is some CSS repetition there and we will fix that, but other than that, the CSS is actually correct, because we are changing the styles of the default platform feedback message.

From our tests, everything is working fine. Are you experiencing any specific issue with the feedback messages? 

Best regards,
Samuel Jesus

no,

suddenly I was wondering why there were no icons, so I delved into the css and found this redundant stuff :)
I know cleaning css is a crappy thing to do :)