> * I don't think there's any need to go through jQuery for this, you should > just be able to set and unset the `disabled` property on `e.target` e.g. > `e.target.disabled = true;` and `e.target.disabled = false;`. If you > absolutely want to use jQuery, I'd suggest $.fn.prop [0] instead of $.fn.attr > > * Fix indentation to stop using tabs, you inserted a tab before the removeAttr > call. bzr can tell you about these by using `bzr cdiff --check-style` when > reviewing your diffs before committing. > > [0] http://api.jquery.com/prop/#prop2
Fixes. bubbling-up events. Used currentTarget instead of target on which current-event listener was attached. -- https://code.launchpad.net/~openerp-dev/openerp-web/6.1bug-817396-vda/+merge/74215 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-web/6.1bug-817396-vda. _______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : [email protected] Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp
