Merge lp:~mwhudson/loggerhead/no-more-mootools into lp:loggerhead

Proposed by Michael Hudson-Doyle
Status: Merged
Merged at revision: not available
Proposed branch: lp:~mwhudson/loggerhead/no-more-mootools
Merge into: lp:loggerhead
To merge this branch: bzr merge lp:~mwhudson/loggerhead/no-more-mootools
Reviewer Review Type Date Requested Status
Paul Hummer (community) Approve
Review via email: mp+3777@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Hi there,

This branch removes mootools and all uses of it from loggerhead, replacing it with YUI3 lurve.

I didn't fully replicate Fx.Slide, instead going for a simple animation of height. I stole the idea of how to compute the height of a display: none element without flickering from whoever did it like this at the epic.

The diff is quite chunky as it basically rewrites all the JS in the project, no way around that really.

Cheers,
mwh

Revision history for this message
Paul Hummer (rockstar) wrote :

Michael-

  This is good work. I noticed a really odd issue when the content of
td.sumcell pushes the content of td.autcell into two lines and so makes all
the table rows two lines high. It's a little funky, not sure if that's
really an issue here though. Could you please file a bug on it? If you need
to reproduce, unfold rev 265 of loggerhead using serve-branches.

  Other than that, I'm good with this branch landing. We talked about the
declarations of functions and the curly brace. It doesn't look like we have a
standard for it right now though, so don't sweat it.

Cheers,
Paul

Revision history for this message
Paul Hummer (rockstar) :
review: Approve
Revision history for this message
Tim Penhey (thumper) wrote :

On Tue, 24 Feb 2009 14:19:17 Paul Hummer wrote:
> Michael-
>
> This is good work. I noticed a really odd issue when the content of
> td.sumcell pushes the content of td.autcell into two lines and so makes all
> the table rows two lines high. It's a little funky, not sure if that's
> really an issue here though. Could you please file a bug on it? If you
> need to reproduce, unfold rev 265 of loggerhead using serve-branches.
>
> Other than that, I'm good with this branch landing. We talked about the
> declarations of functions and the curly brace. It doesn't look like we
> have a standard for it right now though, so don't sweat it.
>
> Cheers,
> Paul

Testing email for me.

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

hi
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkmjTxUACgkQeTTOPm7A7kjY4gCeKfcLCfteS0ReNsqJnRCxto9O
ciwAnRrh7k16zEFelczSdfRtO7128CbM
=Nv46
-----END PGP SIGNATURE-----

Subscribers

People subscribed via source and target branches