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).

Attachments (0)

Change History (6)

Changed August 13, 2010 07:59PM UTC by dmethvin comment:1

component:	support → attributes

Changed October 25, 2010 06:47AM UTC by SlexAxton comment:2

keywords:	cellpadding → cellpadding prop
milestone:	1.4 → 1.5
status:	new → open
type:	bug → enhancement

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

Changed October 25, 2010 06:47AM UTC by SlexAxton comment:3

priority:	major → low

Changed April 15, 2011 03:11AM UTC by timmywil comment:4

version:	1.3.2 → 1.5.2

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

Changed April 15, 2011 03:13AM UTC by timmywil comment:5

pull request

Changed April 16, 2011 04:18PM UTC by john comment:6

milestone:	→ 1.6
resolution:	→ fixed
status:	open → closed

Landed.

Side navigation

#4978 closed enhancement (fixed)