bootstrap-datetimepicker icon indicating copy to clipboard operation
bootstrap-datetimepicker copied to clipboard

Fix Uncaught TypeError: Cannot read property 'top' of undefined

Open uehondor opened this issue 11 years ago • 12 comments

Fixed issue https://github.com/tarruda/bootstrap-datetimepicker/issues/126

uehondor avatar May 09 '13 00:05 uehondor

Was having the same issue! This fixed it!

lamalex avatar May 20 '13 20:05 lamalex

Fixed the issue for me too +1

pgericson avatar May 23 '13 16:05 pgericson

:+1:

I had the same problem and I'm using the js I built from @uehondor branch.

krsmurata avatar May 28 '13 19:05 krsmurata

+1

nicolasbadia avatar Jun 09 '13 14:06 nicolasbadia

:+1:

ppworks avatar Jun 13 '13 13:06 ppworks

good to merge

JulienTant avatar Jul 01 '13 16:07 JulienTant

+1 worked for me

ashleydw avatar Jul 03 '13 12:07 ashleydw

:+1:

pepakriz avatar Jul 22 '13 13:07 pepakriz

:+1: This fixed the issue for me.

senfo avatar Jul 31 '13 14:07 senfo

Merge this, please.

mkoubik avatar Oct 27 '13 12:10 mkoubik

Make sure value changes take effect when clicking away from the input box. Also, make sure to detach the mousedown event when we're done with it.

--- bootstrap-datetimepicker.js 2013-09-05 13:25:50.000000000 -0500
+++ bootstrap-datetimepicker.js 2015-01-09 13:54:46.000000000 -0600
@@ -153,6 +153,7 @@ hide: function() {
         if (collapseData && collapseData.transitioning)
           return;
       }
+      this.change({target: this.$element});
       this.widget.hide();
       this.viewMode = this.startViewMode;
       this.showMode();
@@ -1048,9 +1049,7 @@

     _detachDatePickerGlobalEvents: function () {
       $(window).off('resize.datetimepicker' + this.id);
-      if (!this.isInput) {
-        $(document).off('mousedown.datetimepicker' + this.id);
-      }
+      $(document).off('mousedown.datetimepicker' + this.id);
     },

     _isInFixed: function() {

nhjm449 avatar Jan 09 '15 20:01 nhjm449

It works! thanks

chaintng avatar Oct 07 '15 09:10 chaintng