首页 > 代码库 > UEditor 解决拖拽视频元素改变视频尺寸时,无法保存视频尺寸问题的解决方法
UEditor 解决拖拽视频元素改变视频尺寸时,无法保存视频尺寸问题的解决方法
UEditor虽然强大,但是bug还是蛮多的。比如插入视频元素后,拖拽视频去缩放尺寸,编辑器并没有将实际的尺寸保存下来。当你点击HTML按钮查看源代码时,width和height还是原来的值,再次点击此按钮回到正常状态,缩略图又回到原来的大小了。
翻源代码翻了蛮久,终于把这个问题解决了。问题就出在插入视频后创建视频HTML字符串和HTML字符串与可视化编辑层转化的地方。
当视频上传完成后,创建一个img用于可视化编辑,将默认width和height设置到img的width和height的属性中。当你拖动可视化图片时,就会改变这个img的style中的width和height来改变img的大小(注意,不是改变width和height属性),将新的高和宽设置到。一直到这里其实都没什么问题。
看一下下面这段代码,在ueditor.all.js里:
function creatInsertStr(url,width,height,id,align,classname,type){ var str; switch (type){ case ‘image‘: str = ‘<img ‘ + (id ? ‘id="‘ + id+‘"‘ : ‘‘) + ‘ width="‘+ width +‘" height="‘ + height + ‘" _url="‘+url+‘" class="‘ + classname + ‘"‘ + ‘ src=http://www.mamicode.com/"‘ + me.options.UEDITOR_HOME_URL+‘themes/default/images/spacer.gif" style="background:url(‘+me.options.UEDITOR_HOME_URL+‘themes/default/images/videologo.gif) no-repeat center center; border:1px solid gray;‘+(align ? ‘float:‘ + align + ‘;‘: ‘‘)" />‘>function switchImgAndVideo(root,img2video){ utils.each(root.getNodesByTagName(img2video ? ‘img‘ : ‘embed video‘),function(node){ var className = node.getAttr(‘class‘); var rWidth = node.getStyle("width"); var rHeight = node.getStyle("height"); if(className && className.indexOf(‘edui-faked-video‘) != -1){ //here var html = creatInsertStr( img2video ? node.getAttr(‘_url‘) : node.getAttr(‘src‘),node.getAttr("width"),node.getAttr("height"),null,node.getStyle(‘float‘) || ‘‘,className,img2video ? ‘embed‘:‘image‘); node.parentNode.replaceChild(UE.uNode.createElement(html),node); } if(className && className.indexOf(‘edui-upload-video‘) != -1){ //here var html = creatInsertStr( img2video ? node.getAttr(‘_url‘) : node.getAttr(‘src‘),node.getAttr("width"),node.getAttr("height"),null,node.getStyle(‘float‘) || ‘‘,className,img2video ? ‘video‘:‘image‘); node.parentNode.replaceChild(UE.uNode.createElement(html),node); } }) }这段代码是用于控制html源码与可视化编辑直接的转换的,注释here下一行就是调用上面的方法,问题就出在这里。
这里传入creatInsertStr函数的width和height值是node.getAttr("width|height"),也就是说获取的是标签的width和height属性,而不是style属性里面的width和height。但实际上我们拖动的时候是改变img的style属性,img的width和height不会变,还是默认的值。所以你无论怎么拖动,到最后创建的html代码中video的宽和高都还是默认的。
?所以,我们需要将其统一化,在这里我使他们都用style来设置宽和高,不用width和height属性。修改后的代码如下: ?
function creatInsertStr(url,width,height,id,align,classname,type){ width += ""; height += ""; if(width.indexOf("px") == -1){ width+="px"; } if(height.indexOf("px") == -1){ height+="px"; } var str; switch (type){ case ‘image‘: str = ‘<img ‘ + (id ? ‘id="‘ + id+‘"‘ : ‘‘) + ‘ a="‘+ width +‘" b="‘ + height + ‘" _url="‘+url+‘" class="‘ + classname + ‘"‘ + ‘ src=http://www.mamicode.com/"‘ + me.options.UEDITOR_HOME_URL+‘themes/default/images/spacer.gif" style="background:url(‘+me.options.UEDITOR_HOME_URL+‘themes/default/images/videologo.gif) no-repeat center center; border:1px solid gray;‘+(align ? ‘float:‘ + align + ‘;‘: ‘‘)+‘ width:‘+width+‘; height:‘+height+‘" />‘>? ?我们在传入给creatInsertStr参数时,使用node.getStyle方法获得的元素style中的高和宽,而在creatInsertStr方法中,我们将高和宽设置到style中,并将width和height属性重命名或者删掉,这样,我们就统一了从可视化编辑到源代码所使用的尺寸,这样就可以解决尺寸无法保存的问题。