if (action == "update") {\r
dom.setAttrib(elm, 'cellPadding', cellpadding, true);\r
dom.setAttrib(elm, 'cellSpacing', cellspacing, true);\r
- dom.setAttrib(elm, 'border', border);\r
+\r
+ if (!isCssSize(border)) {\r
+ dom.setAttrib(elm, 'border', border);\r
+ } else {\r
+ dom.setAttrib(elm, 'border', '');\r
+ }\r
+\r
+ if (border == '') {\r
+ dom.setStyle(elm, 'border-width', '');\r
+ dom.setStyle(elm, 'border', '');\r
+ dom.setAttrib(elm, 'border', '');\r
+ }\r
+\r
dom.setAttrib(elm, 'align', align);\r
dom.setAttrib(elm, 'frame', frame);\r
dom.setAttrib(elm, 'rules', rules);\r
if (bordercolor != "") {\r
elm.style.borderColor = bordercolor;\r
elm.style.borderStyle = elm.style.borderStyle == "" ? "solid" : elm.style.borderStyle;\r
- elm.style.borderWidth = border == "" ? "1px" : border;\r
+ elm.style.borderWidth = cssSize(border);\r
} else\r
elm.style.borderColor = '';\r
\r
//elm.outerHTML = elm.outerHTML;\r
\r
inst.nodeChanged();\r
- inst.execCommand('mceEndUndoLevel');\r
+ inst.execCommand('mceEndUndoLevel', false, {}, {skip_undo: true});\r
\r
// Repaint if dimensions changed\r
if (formObj.width.value != orgTableWidth || formObj.height.value != orgTableHeight)\r
html += '<table';\r
\r
html += makeAttrib('id', id);\r
- html += makeAttrib('border', border);\r
+ if (!isCssSize(border)) {\r
+ html += makeAttrib('border', border);\r
+ }\r
+\r
html += makeAttrib('cellpadding', cellpadding);\r
html += makeAttrib('cellspacing', cellspacing);\r
html += makeAttrib('data-mce-new', '1');\r
inst.execCommand('mceInsertContent', false, html);\r
\r
tinymce.each(dom.select('table[data-mce-new]'), function(node) {\r
- var td = dom.select('td', node);\r
+ // Fixes a bug in IE where the caret cannot be placed after the table if the table is at the end of the document\r
+ if (tinymce.isIE && node.nextSibling == null) {\r
+ dom.insertAfter(dom.create('p'), node);\r
+ }\r
\r
+ var tdorth = dom.select('td,th', node);\r
try {\r
- // IE9 might fail to do this selection\r
- inst.selection.select(td[0], true);\r
- inst.selection.collapse();\r
+ // IE9 might fail to do this selection \r
+ inst.selection.setCursorLocation(tdorth[0], 0);\r
} catch (ex) {\r
// Ignore\r
}\r
});\r
\r
inst.addVisual();\r
- inst.execCommand('mceEndUndoLevel');\r
+ inst.execCommand('mceEndUndoLevel', false, {}, {skip_undo: true});\r
\r
tinyMCEPopup.close();\r
}\r
formObj.style.value = dom.serializeStyle(st);\r
}\r
\r
+function isCssSize(value) {\r
+ return /^[0-9.]+(%|in|cm|mm|em|ex|pt|pc|px)$/.test(value);\r
+}\r
+\r
+function cssSize(value, def) {\r
+ value = tinymce.trim(value || def);\r
+\r
+ if (!isCssSize(value)) {\r
+ return parseInt(value, 10) + 'px';\r
+ }\r
+\r
+ return value;\r
+}\r
+\r
function changedBackgroundImage() {\r
var formObj = document.forms[0];\r
var st = dom.parseStyle(formObj.style.value);\r
var st = dom.parseStyle(formObj.style.value);\r
\r
// Update border width if the element has a color\r
- if (formObj.border.value != "" && formObj.bordercolor.value != "")\r
- st['border-width'] = formObj.border.value + "px";\r
+ if (formObj.border.value != "" && (isCssSize(formObj.border.value) || formObj.bordercolor.value != ""))\r
+ st['border-width'] = cssSize(formObj.border.value);\r
+ else {\r
+ if (!formObj.border.value) {\r
+ st['border'] = '';\r
+ st['border-width'] = '';\r
+ }\r
+ }\r
\r
formObj.style.value = dom.serializeStyle(st);\r
}\r
\r
// Add border-width if it's missing\r
if (!st['border-width'])\r
- st['border-width'] = formObj.border.value == "" ? "1px" : formObj.border.value + "px";\r
+ st['border-width'] = cssSize(formObj.border.value, 1);\r
}\r
\r
formObj.style.value = dom.serializeStyle(st);\r