[Citadel Development] Re: Citadel commit log: revision 7691

2009-07-22 Thread IGnatius T Foobar
That appears to have fixed the remaining bugs. Thanks, Matt. Let's all give this a good solid testing before we release it. Everyone please hit this code (the mailbox view particularly) with everything you've got.

[Citadel Development] Citadel commit log: revision 7691

2009-07-22 Thread matt
r7691 | matt | 2009-07-22 21:21:58 -0400 (Wed, 22 Jul 2009) | 3 lines Changed paths: M /trunk/webcit/static/summaryview.js M /trunk/webcit/static/wclib.js Fix drag and drop bugs Fix issue with IE8 -

[Citadel Development] Citadel commit log: revision 7690

2009-07-22 Thread ajc
r7690 | ajc | 2009-07-22 17:35:12 -0400 (Wed, 22 Jul 2009) | 1 line Changed paths: M /trunk/webcit/static/summaryview.js * This change appears to restore ctrl-click's ability to deselect a currently selected message.

[Citadel Development] Re: Citadel commit log: revision 7688

2009-07-22 Thread IGnatius T Foobar
Ok, all yours, Matt ... here's where we stand: * Group select is working * Multi select is partially working. Clicking on an unselected message will select it, but clicking on a selected message will not unselect it. * Hitting "delete" appears to perform the correct server op

[Citadel Development] Citadel commit log: revision 7689

2009-07-22 Thread dothebart
r7689 | dothebart | 2009-07-22 16:25:39 -0400 (Wed, 22 Jul 2009) | 3 lines Changed paths: M /trunk/webcit/graphics.c M /trunk/webcit/roomops.c * fix room logo upload ---

[Citadel Development] Citadel commit log: revision 7688

2009-07-22 Thread ajc
r7688 | ajc | 2009-07-22 15:48:45 -0400 (Wed, 22 Jul 2009) | 1 line Changed paths: M /trunk/webcit/static/summaryview.js * Began debugging the group select code. Several steps in the right direction, but it still needs m

[Citadel Development] Re: Citadel commit log: revision 7687

2009-07-22 Thread IGnatius T Foobar
Aha! Now this is some real progress: WCLog('startMarkingFrom=' + startMarkingFrom + ', finish=' + finish); for(var x = startMarkingFrom; x

[Citadel Development] Re: Citadel commit log: revision 7687

2009-07-22 Thread IGnatius T Foobar
Here's what I've done so far: * Renamed rowId and markedRowId to rowIndex and markedRowIndex, to reflect the fact that we are working with current on-screen index positions rather than the order in which the rows were loaded from the server * Used the expression "parent.rowIndex" t

[Citadel Development] Re: Citadel commit log: revision 7687

2009-07-22 Thread IGnatius T Foobar
Still trying to make sense of this ... what I'm starting to see now is that it attempts to determine the range of messages for group select with row numbers based on the order in which they were received from the server, NOT the order in which they appear on the screen. So, if the table was so

[Citadel Development] Re: Citadel commit log: revision 7687

2009-07-22 Thread IGnatius T Foobar
Ok, there are still some problems in this. I don't think the data model has been fully updated. There are still references to *.ctdlRowId in the code. I updated one of them, and that produced valid starting and ending row numbers, but the group select still didn't work. I also see a re

[Citadel Development] Re: Citadel commit log: revision 7687

2009-07-22 Thread IGnatius T Foobar
Still doesn't work ... I'll see if I can help figure out why.