My favorites | Sign in
Project Home Issues
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 1208: eventDrop issue with jquery 1.7.1
2 people starred this issue and may be notified of changes. Back to list
Status:  Done
Owner:  ----
Closed:  Aug 2013


Sign in to add a comment
 
Reported by gpeschi...@gmail.com, Dec 16, 2011
eventDrop is not trigger with jquery-1.7.1.
Dec 16, 2011
#1 ESW...@gmail.com
I found this bug too and did a bit of debugging. Being a long time programmer but new to javascript I'm taking an educated guess at some of this.

Down in jquery.ui,draggable.js
_trigger: function(type, event, ui) {
}

the "mousedown" type changes to "dragstart" and puts the original event in "originalEvent".

In fullcalendar.js the "function mouse(eve)" call in HoverListener(coordinateGrid) fails because it tries to access ev.PageX which is undefined where as ev.originalEvent.pageX/pageY works.  

Anyway replace mouse(ev) in fallcalendar.js with this to get it to work (at least so far).


       function mouse(ev) {
		var newCell = coordinateGrid.cell(ev.originalEvent.pageX, ev.originalEvent.pageY);
		if (!newCell != !cell || newCell && (newCell.row != cell.row || newCell.col != cell.col)) {
			if (newCell) {
				if (!firstCell) {
					firstCell = newCell;
				}
				change(newCell, firstCell, newCell.row-firstCell.row, newCell.col-firstCell.col);
			}else{
				change(newCell, firstCell);
			}
			cell = newCell;
		}
	}

Dec 30, 2011
#2 mbrew...@utk.edu
Thanks for this, fixed it for me!
Jan 4, 2012
#3 stewart....@gmail.com
Yep +1 for this fix. Works for me too. Great job!
Jan 21, 2012
#4 stenreij...@gmail.com
bugfix for the next version should be:

line(5160) in fullcalendar.js
var newCell = coordinateGrid.cell(ev.pageX, ev.pageY);
With:
var newCell = coordinateGrid.cell(ev.originalEvent.pageX, ev.originalEvent.pageY);

Works perfectly :) Thanks
Jan 27, 2012
#5 oliver.g...@gmail.com
Hi all

Thx for the hint here, but does not work in ie8 and i7 use this line instead and it will:

var newCell = coordinateGrid.cell(ev.originalEvent.pageX || ev.pageX, ev.originalEvent.pageY || ev.pageY);
Feb 5, 2012
#6 adelaide...@gmail.com
Thanks, the last one worked for me perfectly oliver.g!

var newCell = coordinateGrid.cell(ev.originalEvent.pageX || ev.pageX, ev.originalEvent.pageY || ev.pageY);
Feb 8, 2012
#7 wa...@pakgon.com
I've changed jquery-ui.

It's not working when i use jquery-1.7.1.min.js and jquery-ui-1.8.16.custom.min.js.
So, download new one jquery-ui-1.8.17.custom.min.js, it's working


Aug 14, 2013
Project Member #8 adamrs...@gmail.com
with the newest jqui, which fullcalendar now supports, this should work
Status: Done
Labels: -Type-Defect Type-Bug
Sign in to add a comment

Powered by Google Project Hosting