--- Matt Rice <[EMAIL PROTECTED]> wrote: > > --- Richard Frith-Macdonald <[EMAIL PROTECTED]> wrote: > > > > > On 27 Dec 2006, at 15:33, Matt Rice wrote: > > > > > <bug-13916.diff> > > > > > > https://savannah.gnu.org/bugs/?13916 > > > > > > mainly wondering about theming, we could use the > > normal white > > > background and just use the > > disabledControlTextColor > > > or hard code the lightGray color (i'd prefer not > > to for themeing) > > > or add another named color (extension > > disabledControlColor or > > > disabledTextFieldColor something) > > > or other comments on the patch... > > > <bug-13916.diff> > > > > I would have thought that a disabled > > controlBackgroundColor be the > > correct color to use for the background of a > > disabled text field ... > > as if the 'paper' of the field was no longer > there. > > this makes sense to me... > > > > > Using disabledControltextColor would presumably be > > very bad ... as it > > would mean you draw background and text with the > > same color, and make > > the text unreadable. > > > > hmm, i don't see that, controlBackgroundColor is > lightGray and disabledControlTextColor is darkGray > and it seems readable to me (so in fact its no > change > from my patch except using the named color)
oh... i see what you mean, because this is in NSCell, so disabled non NSTextFieldCells will have invisible text... hmm so i guess we need to put this in NSTextFieldCell instead of calling super's drawInteriorWithFrame: for this case __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com _______________________________________________ Gnustep-dev mailing list [email protected] http://lists.gnu.org/mailman/listinfo/gnustep-dev
