Description

In the 1.7 branch, the messages shown above some page were being refactored to go over some theme method add_msg. They now have some class, like 'warning' and show some icon in front of the text message.

The current icons used for it do not have a transparent background, so you see a small white background around the icon, while the real background of that message box is light blue/grey.

Steps to reproduce

See there.

Component selection

Details

MoinMoin Version

1.7 alpha

Discussion

It has to be checked where those icons are used and whether the required change makes sense at any place they are used.

From common.css:

#message .hint {font-style: italic;}
#message .info {
    float: none;
    font-size: 1em;
    color: black;
}
#message .info:before {content: url('../img/icon-info.png'); margin: 0 0.2em;}
#message .warning:before {content: url('../img/alert.png'); margin: 0 0.2em;}
#message .error:before {content: url('../img/icon-error.png'); margin: 0 0.2em;}

From msie.css:

#message .warning { padding-left: 20px; background: url(../img/alert.png) left center no-repeat; }
#message .error { padding-left: 20px; background: url(../img/icon-error.png) left center no-repeat; }

/!\ info is missing for IE!?

Plan


CategoryMoinMoinBugFixed

MoinMoin: MoinMoinBugs/1.7IconsForAddMsgLookBad (last edited 2008-04-26 09:41:30 by ReimarBauer)