Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: 调用 setHtml 方法第一个元素为表格时,表格的样式不一致 #133

Open
flydeemon opened this issue Aug 12, 2024 · 6 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@flydeemon
Copy link

bug 描述

相同的值重复调用 setHtml 方法时,表格的样式不一致

你预期的样子是?

相同的值重复调用 setHtml 方法时,样式要一致

demo 能否复现该 bug ?

能,用官网的例子可以复现,https://cycleccc.github.io/demo/set-html.html

最小成本的复现步骤

<table border="1" cellspacing="0" width="100%">
    <tr>
        <td
            rowspan="2"
            style="
                font-size: 14.67px;
                color: #fed865;
                font-family: 等线;
                vertical-align: middle;
                text-align: center;
                background-color: #44739f;
            "
        >
            <span style="font-size: 14.67px; color: #fed865; font-family: 等线"
                >表格文字</span
            >
        </td>
    </tr>
</table>
  • 步骤:把上面的值输入到demo中,点击 setHtml 按钮,第一次和第二次表格样式会不一致.
image image
@cycleccc cycleccc added the bug Something isn't working label Aug 12, 2024
@cycleccc
Copy link
Owner

已复现,待修复

@cycleccc cycleccc changed the title Bug: 相同的值重复调用 setHtml 方法时,表格的样式不一致 Bug: 调用 setHtml 方法第一个元素为表格时,表格的样式不一致 Aug 12, 2024
@cycleccc
Copy link
Owner

#51
这个 bug 和以上 bug 同源,现在原 wangeditor 和 wangeditor-next 对第一行元素为表格的情况没有很好的解决方法,
暂时的解决方案是在 setHtml 时判断第一个元素是否是 table 如果是 table 则在 第一个元素 html 字符串头部插入一个 <p><br></p>
我后面看看有什么好的解决方案吧,如果有兴趣的话欢迎来尝试解决这个问题.

@hsuna
Copy link
Contributor

hsuna commented Aug 28, 2024

#51 这个 bug 和以上 bug 同源,现在原 wangeditor 和 wangeditor-next 对第一行元素为表格的情况没有很好的解决方法, 暂时的解决方案是在 setHtml 时判断第一个元素是否是 table 如果是 table 则在 第一个元素 html 字符串头部插入一个 <p><br></p> 我后面看看有什么好的解决方案吧,如果有兴趣的话欢迎来尝试解决这个问题.

我试了下 wangeditor ,似乎是没有这个问题的,在官方demo上,新增table时,是不会产生第一行的,但是next版本却会生成多一行

image image

@cycleccc
Copy link
Owner

因为第一行为 table 的时候删不掉,这个我在另一个 issue 写了。还是一样的 bug,table 不能在第一行。

@hsuna
Copy link
Contributor

hsuna commented Aug 28, 2024

因为第一行为 table 的时候删不掉,这个我在另一个 issue 写了。还是一样的 bug,table 不能在第一行。

table的删除是依赖 menu 事件删的吧?我看 v4 也是那样的,可以删除,不提供回车其实不影响交互

@cycleccc
Copy link
Owner

因为第一行为 table 的时候删不掉,这个我在另一个 issue 写了。还是一样的 bug,table 不能在第一行。

table的删除是依赖 menu 事件删的吧?我看 v4 也是那样的,可以删除,不提供回车其实不影响交互

是全选删除的问题, 只要table在第一行就没法全选删除或剪切,只能说两边都很尴尬 , 偏偏这个 bug 也不太好解决

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants