This project is archived and is in readonly mode.

#287 ✓invalid
sorccu

"Events" partially broken in IE (invalid)

Reported by sorccu | August 1st, 2008 @ 11:24 AM | in 2.0 (closed)

"this" in class events seems to be mapped to the $events[type] array in IE. In other browsers it is properly mapped to the Class itself. This ends up breaking at least Drag.Move in IE:


		// Drag.Move.prototype.initialize:
		
		this.addEvent('start', function(){
			this.checkDroppables(); // errors in IE
		}, true);

Here's a simple testcase:


new (new Class({
	Implements: Events,
	initialize: function() {
		this.t = 6;
		this.addEvent('fail', function() { alert($type(this) + '.t = ' + this.t) });
		this.fireEvent('fail');
	}
}));

Results:

"object.t = 6" in good browsers

"array.t = undefined" in IE

Comments and changes to this ticket

  • sorccu

    sorccu August 8th, 2008 @ 12:05 PM

    • Title changed from “"Events" partially broken in IE” to “"Events" partially broken in IE (invalid)”
    • Assigned user cleared.

    I am no longer able to reproduce this, even though at the time I tested this with both IE6 and IE7 and both failed (using an uncompressed 1.2.0-release made with Builder). What actually caused the error remains a mystery, but you may close this ticket for now.

  • Jan Kassens

    Jan Kassens August 9th, 2008 @ 09:34 AM

    • State changed from “new” to “invalid”

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Shared Ticket Bins

People watching this ticket

Tags

Pages