Hi Charles,
Thank you very much for this code. I will add it to the CVS as soon as possible. To get rid of the IE selection, you can try to add row.onselectstart = function() { return false; }; // (untested)
Regards, Thomas
2006/1/19, Charles McNulty charles@charlesmcnulty.com:
Yeah, I experienced the same thing. Not sure what can be done about that if anything.
-Charles
Jon Daley wrote:
This patched worked successfully. It works wonderfully inFirefox, but I tried IE just to see, and it apparently is seeing the left click as part of a highlight command, so the shift click gets processed twice, once by roundcube, and correctly highlights messages in red, but the click is also processed by IE and highlights the text (sometimes the whole page) in blue, as if I wanted to copy and paste the text. It doesn't do it all the time - I had a hard time reproducing it reliably - thought maybe it had to do where I clicked, or whether the shift click was above or below the first click.
On Wed, 18 Jan 2006, Charles McNulty wrote:
Sorry about that. I just retried it and got the same problems. Try this one. I created the one I sent earlier with cvs diff. This one was created with Tortoise with whatever default options it throws into the bag. This one works fine with the latest CVS (1.27 of app.js, I believe.) Good luck. -Charles
Daniel Paquet wrote:
Is the patch work with a freshly cvs'ed source?
The patch is failing for me. With error
patching file app.js Hunk #3 FAILED at 1006. Hunk #5 FAILED at 1043. Hunk #6 FAILED at 1052. Hunk #7 succeeded at 1048 (offset -25 lines). Hunk #8 FAILED at 1053. Hunk #9 succeeded at 1064 (offset -111 lines). Hunk #10 succeeded at 1081 (offset -111 lines). Hunk #11 succeeded at 2762 (offset -111 lines). 4 out of 11 hunks FAILED -- saving rejects to file app.js.rej
Jon Daley http://jon.limedaley.com/
Music: A principal means of glorifying our merciful Creator. -- Henry Peacham