Code review comment for lp:~openerp-dev/openerp-web/7.0-bug-1095246-bth

Revision history for this message
Bhumi Thakkar (Open ERP) (bth-openerp) wrote :

> * Why are you using javascript to set z-index?
> * Why $(ul[0])? (especially when ul probably already a jquery object)
> * Why not fix calendar so it doesn't go through search view?

Fixed calendar instead of ul in searchbox.

review: Needs Resubmitting

« Back to merge proposal