Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 93989 - Background color for the setMessage and setErrorMessage of a WizardPage do not match
Summary: Background color for the setMessage and setErrorMessage of a WizardPage do no...
Status: RESOLVED DUPLICATE of bug 91217
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Billy Biggs CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-06 15:41 EDT by Brent Thurgood CLA
Modified: 2005-05-06 22:34 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Thurgood CLA 2005-05-06 15:41:04 EDT
The background color of the setMesage() and setErrorMessage on a WizardPage do 
not match. The setErrorMessage method of the TitleAreaDialog uses the 
background of the messageLabel for the non-error case, however it uses the 
JFaceColors.getErrorBackground() color for the error case. When the 
setErrorMessage() method is called it causes a gray box to appear within the 
title area on a white background. 

It would be preferred to have the background color of the message and error be 
the same. The error message already contains an error image to notify the user 
of the problem. Changing the color does not look good. 

Snippet from TitleAreaDialog.setErrorMessage()

       if (!showingError) {
                // we were not previously showing an error
                showingError = true;
                // lazy initialize the error background color and image
                if (errorMsgAreaBackground == null) {
                    errorMsgAreaBackground = JFaceColors
                            .getErrorBackground(messageLabel.getDisplay());
                    errorMsgImage = JFaceResources
                            .getImage(DLG_IMG_TITLE_ERROR);
                }
                // show the error
                normalMsgAreaBackground = messageLabel.getBackground();
                setMessageBackgrounds(true);
                messageImageLabel.setImage(errorMsgImage);
                setImageLabelVisible(true);
            }
Within a
Comment 1 Billy Biggs CLA 2005-05-06 22:34:18 EDT
I am going to mark this as a duplicate of bug 91217 since it's related to the
same issue, and I think we should continue the discussion there.
Comment 2 Billy Biggs CLA 2005-05-06 22:34:33 EDT

*** This bug has been marked as a duplicate of 91217 ***