Merge lp:~openerp-dev/openobject-client-web/6.0-opw-19869-cpa into lp:openobject-client-web

Proposed by Chirag Patel (OpenERP)
Status: Merged
Approved by: Olivier Laurent (Open ERP)
Approved revision: 4726
Merge reported by: Olivier Laurent (Open ERP)
Merged at revision: not available
Proposed branch: lp:~openerp-dev/openobject-client-web/6.0-opw-19869-cpa
Merge into: lp:openobject-client-web
Diff against target: 11 lines (+1/-0)
1 file modified
addons/openerp/widgets/form/templates/datetime.mako (+1/-0)
To merge this branch: bzr merge lp:~openerp-dev/openobject-client-web/6.0-opw-19869-cpa
Reviewer Review Type Date Requested Status
Olivier Laurent (Open ERP) (community) Approve
Vaibhav Darji (community) Approve
Review via email: mp+81143@code.launchpad.net

Description of the change

Hello,

Datepicker triggers keyboard / UI lock under IE Internet Explorer

Demo: Project > Task
1. Create new Task.
2. Click on calendar icon to open datepicker in Deadline field.
3. Select a date OR close datepicker without selecting a date

Observed: Can't fill any field in whole form(key event not detach after hide datepicker).

Thanks.

To post a comment you must log in.
Revision history for this message
Vaibhav Darji (vaibhav-openerp) wrote :

Do not do any changes in Calendar Library.
you can do the same code in widget onClose.

Revision history for this message
Vaibhav Darji (vaibhav-openerp) :
review: Needs Fixing
4726. By Chirag Patel (OpenERP)

[FIX] Fixed review code.

Revision history for this message
Vaibhav Darji (vaibhav-openerp) :
review: Approve
Revision history for this message
Olivier Laurent (Open ERP) (olt) wrote :

I could reproduce the bug under IE. The issue is fixed by your patch.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'addons/openerp/widgets/form/templates/datetime.mako'
--- addons/openerp/widgets/form/templates/datetime.mako 2011-01-11 14:53:37 +0000
+++ addons/openerp/widgets/form/templates/datetime.mako 2011-11-09 11:27:48 +0000
@@ -18,6 +18,7 @@
18 showsTime: ${str(picker_shows_time).lower()},18 showsTime: ${str(picker_shows_time).lower()},
19 onClose: function(cal){19 onClose: function(cal){
20 cal.destroy();20 cal.destroy();
21 cal.hide();
21 }22 }
22 });23 });
23 });24 });