tinyMCEPopup.restoreSelection();\r
\r
if (!AutoValidator.validate(formObj)) {\r
- tinyMCEPopup.alert(inst.getLang('invalid_data'));\r
+ tinyMCEPopup.alert(AutoValidator.getErrorMessages(formObj).join('. ') + '.');\r
return false;\r
}\r
\r
// Get form data\r
cols = formObj.elements['cols'].value;\r
rows = formObj.elements['rows'].value;\r
- border = formObj.elements['border'].value != "" ? formObj.elements['border'].value : 0;\r
+ border = formObj.elements['border'].value != "" ? formObj.elements['border'].value : 0;\r
cellpadding = formObj.elements['cellpadding'].value != "" ? formObj.elements['cellpadding'].value : "";\r
cellspacing = formObj.elements['cellspacing'].value != "" ? formObj.elements['cellspacing'].value : "";\r
align = getSelectValue(formObj, "align");\r
\r
// Update table\r
if (action == "update") {\r
- inst.execCommand('mceBeginUndoLevel');\r
-\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
capEl = elm.ownerDocument.createElement('caption');\r
\r
if (!tinymce.isIE)\r
- capEl.innerHTML = '<br _mce_bogus="1"/>';\r
+ capEl.innerHTML = '<br data-mce-bogus="1"/>';\r
\r
elm.insertBefore(capEl, elm.firstChild);\r
}\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('_mce_new', '1');\r
+ html += makeAttrib('data-mce-new', '1');\r
\r
if (width && inst.settings.inline_styles) {\r
if (style)\r
\r
if (caption) {\r
if (!tinymce.isIE)\r
- html += '<caption><br _mce_bogus="1"/></caption>';\r
+ html += '<caption><br data-mce-bogus="1"/></caption>';\r
else\r
html += '<caption></caption>';\r
}\r
\r
for (var x=0; x<cols; x++) {\r
if (!tinymce.isIE)\r
- html += '<td><br _mce_bogus="1"/></td>';\r
+ html += '<td><br data-mce-bogus="1"/></td>';\r
else\r
html += '<td></td>';\r
}\r
\r
html += "</table>";\r
\r
- inst.execCommand('mceBeginUndoLevel');\r
-\r
// Move table\r
if (inst.settings.fix_table_elements) {\r
var patt = '';\r
} else\r
inst.execCommand('mceInsertContent', false, html);\r
\r
- tinymce.each(dom.select('table[_mce_new]'), function(node) {\r
- var td = dom.select('td', node);\r
-\r
- inst.selection.select(td[0], true);\r
- inst.selection.collapse();\r
+ tinymce.each(dom.select('table[data-mce-new]'), function(node) {
+ var tdorth = dom.select('td,th', 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
+ try {\r
+ // IE9 might fail to do this selection \r
+ inst.selection.setCursorLocation(tdorth[0], 0);\r
+ } catch (ex) {\r
+ // Ignore\r
+ }\r
\r
- dom.setAttrib(node, '_mce_new', '');\r
+ dom.setAttrib(node, 'data-mce-new', '');\r
});\r
\r
inst.addVisual();\r
- inst.execCommand('mceEndUndoLevel');\r
+ inst.execCommand('mceEndUndoLevel', false, {}, {skip_undo: true});\r
\r
tinyMCEPopup.close();\r
}\r
\r
var cols = 2, rows = 2, border = tinyMCEPopup.getParam('table_default_border', '0'), cellpadding = tinyMCEPopup.getParam('table_default_cellpadding', ''), cellspacing = tinyMCEPopup.getParam('table_default_cellspacing', '');\r
var align = "", width = "", height = "", bordercolor = "", bgcolor = "", className = "";\r
- var id = "", summary = "", style = "", dir = "", lang = "", background = "", bgcolor = "", bordercolor = "", rules, frame;\r
+ var id = "", summary = "", style = "", dir = "", lang = "", background = "", bgcolor = "", bordercolor = "", rules = "", frame = "";\r
var inst = tinyMCEPopup.editor, dom = inst.dom;\r
var formObj = document.forms[0];\r
var elm = dom.getParent(inst.selection.getNode(), "table");\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