Description Mike Wilson 2001-10-11 14:22:04 EDT

DLW (6/20/2001 4:36:38 PM)
		Native label widgets often show a grayed out appearance when disabled.  I think this
		would be a very useful feature to have in CLabel.

NOTES:

	DLW (6/20/2001 4:37:25 PM)
		I have added this functionality to CLabel in my workspace.  You might want to change it 
		to add an API to set the disabled background and foreground colors, but for now, I just use 
		the native colors.

		In the method onPaint(), I have made the following change to the section where the
		text is drawn to the GC:

	// draw the text
	if (t != null) {
		int textHeight = gc.getFontMetrics().getHeight();
		Display disp = getDisplay();
		if (isEnabled()) {
			gc.setForeground(getForeground());
		} else {
			// draw the text as "grayed out"
			gc.setForeground(disp.getSystemColor(SWT.COLOR_WIDGET_HIGHLIGHT_SHADOW));
			gc.drawText(t, x + 1, rect.y + 1 + (rect.height-textHeight)/2, true);
			gc.setForeground(disp.getSystemColor(SWT.COLOR_WIDGET_NORMAL_SHADOW));
		}
		gc.drawText(t, x, rect.y + (rect.height-textHeight)/2, true);
	}

		In addition, I added the following method so that the text is repainted when the
		enablement is changed:


/*
 * Set the label's enablement to <code>enabled</code>.
 * Redraw the widget so the text gets redrawn to reflect the new value.
 */
public void setEnabled(boolean enabled) {
	if (enabled != isEnabled()) {
		super.setEnabled(enabled);
		redraw();
	}
}

SN (6/20/01 5:06:12 PM)
	This "(enabled != isEnabled())" is wrong.  It should be "(enabled != getEnabled())"
	or don't even do the test, just redraw.  The "draw disabled" code is also Windows
	specific ... but given that we don't have any API to support drawing text disabled,
	there's not much we can do right now (this API exists for Images).

DLW (6/21/2001 3:30:50 pm)
	How is it Windows specific?  Those constants not available on other SWT flavors.
	Or are you saying it b/c other platforms don't "gray out" their labels when disabled?
	Or that they don't have an "etched" appearance?

	McQ (26/06/2001 9:13:35 AM) -
		VI just mentioned that you are using this to do the AWT label. You need to
		watch out when using the "custom" widgets in the AWT implementation. The
		Eclipse team gets to decide what the L&F of the custom widgets is. This
		could cause you grief if they decide to change it radically.

	DLW (6/27/2001 4:56:16 pm)
		It would be helpful if the comment for getEnabled() vs. isEnabled() explained the
		difference between the two, and when you would use one vs. the other.  Currently 
		they have an identical comment.