Mohammed Shekha(Open ERP) has proposed merging
lp:~openerp-dev/openerp-web/6.1-opw-572495-msh into lp:openerp-web/6.1.
Requested reviews:
OpenERP Core Team (openerp)
For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-572495-msh/+merge/96732
Hello,
Fixed the issue of datetime widget which is not hide when it is clicked two
times.
Demo :- Click on the open datetime widget image consecutively two times without
selecting date or time, you will see the jQuery datetimepicker widget will not
be hidden, it is stayed open and only going to hidden when we refresh whole
page.
Reason :- Here we are chekcing whether datetimepicker widget is visible or not
in view_form.js, if !self.picker('widget') then we are showing the
input_picker, but here the self.picker('widget') is giving wrong reference
element.
(The issue can also be fixed by checking
self.$input_picker['datepicker']("widget").is(':visible'))
Thanks.
--
https://code.launchpad.net/~openerp-dev/openerp-web/6.1-opw-572495-msh/+merge/96732
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openerp-web/6.1-opw-572495-msh.
=== modified file 'addons/web/static/lib/jquery.ui.timepicker/js/jquery-ui-timepicker-addon.js'
--- addons/web/static/lib/jquery.ui.timepicker/js/jquery-ui-timepicker-addon.js 2012-02-14 10:51:45 +0000
+++ addons/web/static/lib/jquery.ui.timepicker/js/jquery-ui-timepicker-addon.js 2012-03-09 09:32:21 +0000
@@ -913,7 +913,7 @@
tmp_args = arguments;
if (typeof(o) == 'string'){
- if(o == 'getDate')
+ if(o == 'getDate' || o == 'widget')
return $.fn.datepicker.apply($(this[0]), tmp_args);
else
return this.each(function() {
_______________________________________________
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
