Bug Tracker

Ticket #3017 (new bug)

Opened 7 months ago

Last modified 7 months ago

v1.2.6 Incompatibility with jqModal

Reported by: Myrlyn Assigned to: jdsharp
Type: bug Priority: major
Milestone: 1.3 Component: plugin
Version: 1.2.6 Keywords: jqModal, v1.2.6, 1.2.6
Cc: doug.domeny@ektron.com Needs: Test Case

Description

I've been using the jqModal for a few months now, and recently updated my copy of the jQuery library to v1.2.6. After upgrading I noticed that the z-index of the overlay isn't properly implimented. The effect is that the overlay only partially obscures elements within the page in IE. Most notably, form fields elements shine through and can be interacted with.

Rolling back to my older version of jQuery (v1.2.3), enables the plugin to work properly.

Attachments

jqModalTest.zip (33.7 kB) - added by Myrlyn 7 months ago.
jqModal bug Test Files with v1.2.6

Change History

  Changed 7 months ago by flesler

  • owner set to jdsharp
  • component changed from core to plugin

Jonathan: I assign it to you so you can check that. If you find a bug in jQuery, let me know.

  Changed 7 months ago by flesler

  • need changed from Review to Test Case

Could you provide a test case ? a minimalistic html file with the requires html and js to reproduce the problem. Thanks.

in reply to: ↑ description   Changed 7 months ago by doug

Replying to Myrlyn:

Tracked down the problem... Needed to re-add " || undefined" from 1.2.3 to the end of this code snippet:

attr: function( name, value, type ) {
	var options = name;
	// Look for the case where we're accessing a style value
	if ( name.constructor == String )
		if ( value === undefined )
			return this[0] && jQuery[ type || "attr" ]( this[0], name ) || undefined;

-Doug, co-worker with Myrlyn at Ektron

Changed 7 months ago by Myrlyn

jqModal bug Test Files with v1.2.6

  Changed 7 months ago by Myrlyn

Here is a quick sample page you can use to see the defect in IE (we were testing with IE7). Doug was able to do some debugging this morning and correct the defect for our current copy of v1.2.6, but if you can integrate the necessary change into a new version of jQuery it would be appreciated.

Thanks, Keith

Note: See TracTickets for help on using tickets.