Re: [Mixxx-devel] MIDI control of playlist / loading into left/right player / need a pointer

2007-12-13 Thread Garth Dahlstrom
er... sorry, been starring at Herc USB / HID code too long In MIDI under Win32, Herc sends 0 and 127 (7F). so (v==127) is what we want. On Dec 13, 2007 1:56 PM, Jan Jockusch <[EMAIL PROTECTED]> wrote: > Garth Dahlstrom wrote: > > > > +if (v > 1.0) m_pView->m_pTrackTableView->moveNext(); >

Re: [Mixxx-devel] MIDI control of playlist / loading into left/right player / need a pointer

2007-12-13 Thread Jan Jockusch
Garth Dahlstrom wrote: > > +if (v > 1.0) m_pView->m_pTrackTableView->moveNext(); > > Those should be (v >= 1.0) or just (v), right? > > ( Pretty sure Herc's send only 0 (up) and 1 (down) for buttons nothing > more 1... ) > > Cheers, > > -G Well, I had a reason for this, of course. When testing

Re: [Mixxx-devel] MIDI control of playlist / loading into left/right player / need a pointer

2007-12-13 Thread Garth Dahlstrom
+if (v > 1.0) m_pView->m_pTrackTableView->moveNext(); Those should be (v >= 1.0) or just (v), right? ( Pretty sure Herc's send only 0 (up) and 1 (down) for buttons nothing more 1... ) Cheers, -G On Dec 13, 2007 9:26 AM, Jan Jockusch <[EMAIL PROTECTED]> wrote: > Hi Albert, > > Hey Jan, > >

Re: [Mixxx-devel] MIDI control of playlist / loading into left/right player / need a pointer

2007-12-13 Thread Adam Davison
Hi Jan, I don't know this part of Qt very well myself but I would imagine there is a function like setSelectedIndex() that you want to use. moveCursor() is probably protected because it's the function that's implemented to catch real cursor motion events so overriding it probably isn't what you wa

Re: [Mixxx-devel] MIDI control of playlist / loading into left/right player / need a pointer

2007-12-13 Thread Jan Jockusch
Hi Albert, Hey Jan, I'm just in the middle of my exam period so I don't have enough time to fully review your patch yet, but for now, have you tried using this: moveCursor(QAbstractItemView::MovePrevious, Qt::NoModifier); (ie. MovePrevious instead of MoveUp, etc.) Looks good though, hopefully