Review: Needs Fixing

* Most of the configuration keys in the listview (all of them apart from header 
on/off) use an "-able" form (selectable, deletable, addable, sortable, 
reorderable) so I think this one should as well (although I'm not quite sure 
what the word would be)

* Event#toElement is a microsoft property (though others may have implemented 
it, notsure). It does not seem implemented by Firefox, in any case. Should use 
`e.target`: http://jsfiddle.net/gJ6Au/ (works in Safari, Firefox and Opera, not 
tested in MSIE)
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-listview-pencil-fme/+merge/85714
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-listview-pencil-fme.

_______________________________________________
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

Reply via email to