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 = formObj.elements['align'].options[formObj.elements['align'].selectedIndex].value;\r
- frame = formObj.elements['frame'].options[formObj.elements['frame'].selectedIndex].value;\r
- rules = formObj.elements['rules'].options[formObj.elements['rules'].selectedIndex].value;\r
+ align = getSelectValue(formObj, "align");\r
+ frame = getSelectValue(formObj, "tframe");\r
+ rules = getSelectValue(formObj, "rules");\r
width = formObj.elements['width'].value;\r
height = formObj.elements['height'].value;\r
bordercolor = formObj.elements['bordercolor'].value;\r
bgcolor = formObj.elements['bgcolor'].value;\r
- className = formObj.elements['class'].options[formObj.elements['class'].selectedIndex].value;\r
+ className = getSelectValue(formObj, "class");\r
id = formObj.elements['id'].value;\r
summary = formObj.elements['summary'].value;\r
style = formObj.elements['style'].value;\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
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
html += makeAttrib('border', border);\r
html += makeAttrib('cellpadding', cellpadding);\r
html += makeAttrib('cellspacing', cellspacing);\r
+ html += makeAttrib('data-mce-new', '1');\r
\r
if (width && inst.settings.inline_styles) {\r
if (style)\r
style += '; ';\r
\r
+ // Force px\r
+ if (/^[0-9\.]+$/.test(width))\r
+ width += 'px';\r
+\r
style += 'width: ' + width;\r
} else\r
html += makeAttrib('width', width);\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
- inst.execCommand('mceInsertContent', false, html);\r
+ // Move table\r
+ if (inst.settings.fix_table_elements) {\r
+ var patt = '';\r
+\r
+ inst.focus();\r
+ inst.selection.setContent('<br class="_mce_marker" />');\r
+\r
+ tinymce.each('h1,h2,h3,h4,h5,h6,p'.split(','), function(n) {\r
+ if (patt)\r
+ patt += ',';\r
+\r
+ patt += n + ' ._mce_marker';\r
+ });\r
+\r
+ tinymce.each(inst.dom.select(patt), function(n) {\r
+ inst.dom.split(inst.dom.getParent(n, 'h1,h2,h3,h4,h5,h6,p'), n);\r
+ });\r
+\r
+ dom.setOuterHTML(dom.select('br._mce_marker')[0], html);\r
+ } else\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
+\r
+ try {\r
+ // IE9 might fail to do this selection\r
+ inst.selection.select(td[0], true);\r
+ inst.selection.collapse();\r
+ } catch (ex) {\r
+ // Ignore\r
+ }\r
+\r
+ dom.setAttrib(node, 'data-mce-new', '');\r
+ });\r
+\r
inst.addVisual();\r
inst.execCommand('mceEndUndoLevel');\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
style = dom.serializeStyle(st);\r
dir = dom.getAttrib(elm, 'dir');\r
lang = dom.getAttrib(elm, 'lang');\r
- background = getStyle(elm, 'background', 'backgroundImage').replace(new RegExp("url\\('?([^']*)'?\\)", 'gi'), "$1");\r
+ background = getStyle(elm, 'background', 'backgroundImage').replace(new RegExp("url\\(['\"]?([^'\"]*)['\"]?\\)", 'gi'), "$1");\r
formObj.caption.checked = elm.getElementsByTagName('caption').length > 0;\r
\r
orgTableWidth = width;\r
\r
// Update form\r
selectByValue(formObj, 'align', align);\r
- selectByValue(formObj, 'frame', frame);\r
+ selectByValue(formObj, 'tframe', frame);\r
selectByValue(formObj, 'rules', rules);\r
selectByValue(formObj, 'class', className, true, true);\r
formObj.cols.value = cols;\r
var st = dom.parseStyle(formObj.style.value);\r
\r
if (st['background-image'])\r
- formObj.backgroundimage.value = st['background-image'].replace(new RegExp("url\\('?([^']*)'?\\)", 'gi'), "$1");\r
+ formObj.backgroundimage.value = st['background-image'].replace(new RegExp("url\\(['\"]?([^'\"]*)['\"]?\\)", 'gi'), "$1");\r
else\r
formObj.backgroundimage.value = '';\r
\r