Jonathan -

Your code is looking good. The only line that looked strange to me was:
$(ul).append('</iframe>').prepend('<iframe style="position: absolute;
z-index: -1;">');

I think you might be looking for a "wrapInner" equivalent:
http://motherrussia.polyester.se/jquery/wrapinner/

--John

On 2/2/07, Jonathan Sharp <[EMAIL PROTECTED]> wrote:
> I'm just starting on a refactor of this code and am making several
> improvements including accepting settings when initializing, handling events
> through delegation, replacing the iframe code with bgiframe and supporting
> vertical menus.
>
> So before I undergo this refactor. There have been several approaches to
> structuring plugin's and I wanted to ask which approach is "standard" or
> closest to "standard" for how plugin's should be developed. A few things to
> note is the use of the wrapper function passing in jQuery, internal
> functions not bound directly to the jQuery object. So any feedback is much
> appreciated!
>
> Source Reference:
> http://jdsharp.us/code/jQuery/plugins/jdMenu/jquery.jdMenu.js
>
> Cheers,
> -Jonathan
> _______________________________________________
> jQuery mailing list
> [email protected]
> http://jquery.com/discuss/
>
>
>

_______________________________________________
jQuery mailing list
[email protected]
http://jquery.com/discuss/

Reply via email to