Re: NSMatrix keyboard stuff

[Matt Rice]

On 2007-01-10 09:17:49 -0800 Matt Rice <address@hidden> wrote:

> On 2007-01-10 09:04:57 -0800 Matt Rice <address@hidden> wrote:
>
> > On 2007-01-10 07:59:16 -0800 Fred Kiefer <address@hidden> wrote:
> >
> > > HI Matt,
> > >
> > > could you please explain this patch a bit? Only the last bit is 
> > > about
> > > selecting the right cell in NSTrackModeMatrix. Are the other change 
> > > a
> > > result of this?
> > > It looks like you intent to prevent the setting of the cell state 
> > > for
> > > this mode. Is this correct?
> >
> > Yes, you understand the patch correctly,
> >
> > I've just noticed the difference between -keyCell and -selected*,
> >    which seems to be what i need/the reason for my confusion.
> > this one is much cleaner :D
>
> ok... so this one isn't going to be correct either.. because 
> _selectCellAtRow:column: sets the state..

alright sorry for the barrage of emails.
I hadn't noticed the above because _selectCell:atRow:column: sets it 
to NSOnState.
and the button i tested it with was on, so it was set to On then to 
Off by the setNextState.

_selectCell:atRow:column: isn't right because the selected cell can 
have an NSOffState.
for track/highlight mode and NSOnState for list/radio...

hopefully this patch takes care of it :D

<matrix-kbd3.diff>

matrix-kbd3.diff
Description: Text document