[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: gui changes.

From: Nicola Pero
Subject: Re: gui changes.
Date: Fri, 23 Mar 2001 16:59:42 +0000 (GMT)

> > Read the doc for NSCell -setShowsFirstResponder:
> > 
> > "Sets whether the receiver displays a dotted-line outline when it
> > assumes first responder status."
> > 
> > That's a heap of rubbish.  A cell is *not* a responder, and so it can
> > *never* assume first responder status.  The fact that a cell is not a
> It seems to me a documentation bug.

Ok - fine then.

> > The correct totally natural and obvious design is - 
> > 
> > a cell should have a state variable which determines whether it draws
> > a dotted outline or not.  This variable can be set/read by using
> > something like -setDrawsDottedOutline:, -drawsDottedOutline.
> This is the behaviour of -setShowsFirstResponder: and -showsFirstResponder

Ok - that's fine then - it looks nice - then you must fix NSCell
-setShowsFirstResponder: on CVS, because that is not what you implemented.

> > The cell
> > takes *no* decision whatsoever about responders; it just implements
> > the code to draw the dotted outline, and that code can be turned on
> In the -NSCell drawInterior... it does not check anymore if the control
> cell is the first responder... this is a 'bug' that I have to commit.

`anymore`...  Anymore on your machine perhaps.  On CVS it *does* check.

> > I suggest using setDrawsDottedOutline: drawsDottedOutline in our
> > implementation.  I suggest implementing setShowsFirstResponder: in the
> > control.  And finally, to attempt some sort of portability, we
> What you want to to is basically change the method name 
> -setShowsFirstResponder
> to -setDrawsDottedOutline: and this does not look reasonable to me.

If the -setShowsFirstResponder: method does what my hypotethical
setDrawsDottedOutline: would do, and not what the apple documentation
says, nor what your implementation on CVS does, then it's fine to keep the
name it has.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]