Overall, this branch looks good. There is a slight bug in the
implementation of showing the thumbnail that should probably be fixed
before this lands (detailed below). Other than that, I'm happy to merge
this for you.
There's an issue here with the thumbnail popup. If the image pops up
under your mouse, it flickers on and off because of the mouseOut event.
I'm not sure the best way to fix this, but you'll probably need to do
some checking on the mouse position, and see if they overlap, and move
the document.removeChild() to the mouseOut event of image if it's
visible under the mouse. Complicated, I know.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Overall, this branch looks good. There is a slight bug in the
implementation of showing the thumbnail that should probably be fixed
before this lands (detailed below). Other than that, I'm happy to merge
this for you.
review needs_fixing
Paul
> === modified file 'lifestream.html' timetd) ; createElement( 'img'); function( ){ style.position = "absolute"; body.appendChil d(this. img); body.removeChil d(this. img); createElement( "td"); ld(document. createTextNode( src));
> --- lifestream.html 2008-12-12 00:16:45 +0000
> +++ lifestream.html 2008-12-27 19:16:42 +0000
> @@ -168,6 +172,19 @@ ls = {
> timetd.className = "time";
> tr.appendChild(
> td.className = "data";
> + if(thumbnail) {
> + td.img = document.
> + td.img.src = thumbnail ;
> +
> + $(td).hover(
> + this.img.
> + this.img.style.top = (this.offsetTop + this.offsetHeight) + "px";
> + this.img.style.left = this.offsetLeft + "px";
> + document.
> + }, function(){
> + document.
> + });
> + }
> tr.appendChild(td);
> var srctd = document.
> srctd.appendChi
>
There's an issue here with the thumbnail popup. If the image pops up removeChild( ) to the mouseOut event of image if it's
under your mouse, it flickers on and off because of the mouseOut event.
I'm not sure the best way to fix this, but you'll probably need to do
some checking on the mouse position, and see if they overlap, and move
the document.
visible under the mouse. Complicated, I know.
-----BEGIN PGP SIGNATURE----- enigmail. mozdev. org
Wh2sACgkQHE2KxY Yv8I+k6ACgmqkSq ylWW45wuGC1kbvo upuC f5Mb2CfoVwPYK2m Oa
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAkl
xTkAn06099c2nOo
=VSPM
-----END PGP SIGNATURE-----