openerp-dev-web team mailing list archive

["Sananaz \(Open ERP\)" <sma@xxxxxxxxxxx>]

> * Not only are jQuery's .addClass(String), .removeClass(String) and
> .attr(args) chainable, *they can operate on jQuery element sets*. There is no
> reason to use an explicit iteration loop to just set the same classes and
> attributes on *all* the elements of a collection
> 

Thanks xavier, yes no need to use explicit iteration loop :(.

> * At the JS level, use boolean values for things like ``disabled`` (as well as
> for ``readonly``): .attr('disabled', true) and .attr('disabled', false)
> 

I have change this 'disabled' replace with just `true`, `false` conditions.

> * jQuery already has a pseudo-class for the selector "input, textarea, select,
> button": ":input" http://api.jquery.com/input-selector/ you should use that
> instead of doing it by hand.

Also change this. Now use the jQuery selector.


Thank you.


-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-4916-sma/+merge/55889
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-4916-sma.