[jQuery] Re: How to reduce this code by chain them up?

[Liming]

Ah.. okay. so it's .html() that's messing me up.

Thanks Ricardo.  My original way of querying the templateTable by
passing itself as a "context" was awkward, the way you did is much
more "fluent".

That helps me a lot not only in this particular example, but helped me
understand the chainability as well. I greatly appreciate your help!!

Thanks again.


On Apr 30, 1:01 pm, Ricardo <ricardob...@gmail.com> wrote:
> On Apr 30, 10:27 am, Liming <lmxudot...@gmail.com> wrote:
>
>
>
> > Hello,
>
> > I have a div ( id="rightheader")  and inside there is a table. The
> > following code replace all content of the table with empty space and
> > then remove the table header.
>
> > var $templateTable = $("div#rightheader").clone();
> > $("tbody > tr > td > div ", $templateTable).html("&nbsp");
> > $("thead",$templateTable).remove();
>
> > I feel it might be unnecessary, but I can't figure out how to chain up
> > the statements.
>
> > I tried
>
> > var $templateTable = $("div#rightheader")
> >                                  .clone()
> >                                 .("tbody > tr > td > div ").html("&nbsp")
> >                                 .("thead").remove();
>
> > Got an error
>
> > then tried
>
> >         var $templateTable = $("div#rightheader")
> >                                                                  .clone()
> >                                                                  
> > .find("tbody > tr > td > div ").html("&nbsp")
> >                                                                  
> > .find("thead").remove();
>
> > but that just doesn't produce anything.
>
> > Sorry, I'm a real beginner on JQuery, any pointer is greatly
> > appreciate it.
>
> > Thanks
>
> the html() method doesn't return an object, so it has to be last:
>
> $("div#rightheader").clone()
>     .find("thead").remove().end()
>     .find("tbody > tr > td > div ").html("&nbsp");
>
> In this case I'd say chaining makes it more confusing, as you're
> copying/destroying elements on the way, not just traversing it. It's
> just a line shorter than this, which will be much clearer as I assume
> you're further modifying $templateTabe later:
>
> var $templateTable = $("div#rightheader").clone();
>
> $templateTable
>     .find("thead").remove().end()
>     .find("tbody > tr > td > div ").html("&nbsp");