#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 11 years ago by
| Component: | support → attributes |
|---|
comment:2 Changed 11 years ago by
| Keywords: | prop added |
|---|---|
| Milestone: | 1.4 → 1.5 |
| Status: | new → open |
| Type: | bug → enhancement |
comment:3 Changed 11 years ago by
| Priority: | major → low |
|---|
comment:4 Changed 11 years ago by
| Version: | 1.3.2 → 1.5.2 |
|---|
I think we can support this in 1.6. It's a one-line addition. Anyone opposed?
comment:6 Changed 11 years ago by
| Milestone: | → 1.6 |
|---|---|
| Resolution: | → fixed |
| Status: | open → closed |
Landed.
Note: See
TracTickets for help on using
tickets.
Adding this as an enhancement since jQuery doesn't necessarily claim support for this.