Can't insert column after last column in IE

[anonymous]

Just contributing a suggested patch here. If you try to insert a new column after the last column in a table, IE returns an error. The solution is to omit the 'ref' parameter when the current cell is the last one in the row:

            case "TO-col-insert-before":
	    case "TO-col-insert-after":
		var td = this.getClosest("td");
		if (!td) {
			break;
		}
		var rows = td.parentNode.parentNode.rows;
		var index = td.cellIndex;
                // added line:
                var lastColumn = (td.parentNode.cells.length == index + 1);
		for (var i = rows.length; --i >= 0;) {
			var tr = rows[i];
			var otd = editor._doc.createElement("td");
			otd.innerHTML = mozbr;
			//added block:
                        if (lastColumn && HTMLArea.is_ie) {
				tr.insertBefore(otd);
			} else {
				var ref = tr.cells[index + (/after/.test(button_id) ? 1 : 0)];
				tr.insertBefore(otd, ref);
			}
		}

Note that the line 'var ref = ' was moved

[gogo]

changeset:413

[gogo]

changeset:413

[guest]

i tried to insert a column before the last column in IE 7 (don't know about IE 6), and it's not working, it inserts after. when with Firefox it works great.

working with today's download of Xinha_0.931.zip

[wymsy]

It works correctly for me in IE 6. I don't have IE 7 so can't test that.

[ray]

I can confirm it inserts the column after in IE6 & 7

[gogo]

So I think that means it's fixed.