Bug Tracker

Opened 8 years ago

Closed 6 years ago

Last modified 5 years ago

#4978 closed enhancement (fixed)

jQuery.prop missing cellpadding?

Reported by: janzich Owned by:
Priority: low Milestone: 1.6
Component: attributes Version: 1.5.2
Keywords: cellpadding prop Cc:
Blocked by: Blocking:

Description

I'm not entirely familiar with the purpose of jQuery.prop, but I'm guessing its purpose is to fix some attributes to their standard form. It seems that it's missing cellPadding (cellSpacing is already there) which causes a problem in IE7 (tested in IE8 compatibility mode).

The following snippet in IE7 sets cellspacing, but does not set cellpadding:

$("<table/>").attr("cellspacing", "10").attr("cellpadding", "10");

The following works:

$("<table/>").attr("cellspacing", "10").attr("cellPadding", "10");

The problem is caused by the case-sensitivity of the setAttribute function in pre-IE8 (documented in MSDN).

Change History (6)

comment:1 Changed 7 years ago by dmethvin

Component: supportattributes

comment:2 Changed 7 years ago by SlexAxton

Keywords: prop added
Milestone: 1.41.5
Status: newopen
Type: bugenhancement

Adding this as an enhancement since jQuery doesn't necessarily claim support for this.

comment:3 Changed 7 years ago by SlexAxton

Priority: majorlow

comment:4 Changed 6 years ago by timmywil

Version: 1.3.21.5.2

I think we can support this in 1.6. It's a one-line addition. Anyone opposed?

comment:5 Changed 6 years ago by timmywil

comment:6 Changed 6 years ago by john

Milestone: 1.6
Resolution: fixed
Status: openclosed

Landed.

Note: See TracTickets for help on using tickets.