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

feat: 升级到25.0.0 #128

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from
Open

Conversation

chenEdgar
Copy link
Contributor

@chenEdgar chenEdgar commented Feb 23, 2021

Why

目前组件使用的是16.0.0版本,存在以下问题需要解决

  1. Url边缘识别 Feature: link 文本在边缘处自动跳出编辑区 #71
  2. 上传图片后,不能在图片前面插入空行 上传图片后,不能在图片前面插入空行 #86

How

  1. 升级@ckeditor/*相关依赖到最新版本
  2. 解决升级后,api调用相关bug

Test

  1. 原有测试demo功能正常(人肉测试)
  2. URL边缘判断问题 Feature: link 文本在边缘处自动跳出编辑区 #71
    修改前: 插入URL后,URL后面输入的文字(下图空格)仍然被当做url的文本了
    image

修改后
image

  1. 上传图片后,不能在图片前面插入空行 上传图片后,不能在图片前面插入空行 #86
    修改前:
    image

修改后:
image

其他

本次升级,修复了“�存在图片时不能全选的问题” #122 。解决方案是在plugin/Essentials.js 文件引入 import SelectAll from '@ckeditor/ckeditor5-select-all/src/selectall' 指令

重要

本次升级后,丢掉了针对中文输入法输入的修复代码 #81 。因此issue #81 的问题将再现

丢弃原因:原有的修复是在组合是阻止了渲染?但带来了以下问题。

  1. 对于中文格式化文本,输入后不符合预期。

复现步骤:
Mac 11.1, 搜狗智慧输入法(拼音)

  1. �在富文本中输入分别 1,2,3换行,
  2. 操作栏选中加粗
  3. 在新的行输入字体
  4. 按enter后,光标重定向到首行第一个字符前,且输入的中文丢失,只输入了中文的首个拼音字母
    image

Docs

文档没有变更

@auto-add-label auto-add-label bot added the enhancement New feature or request label Feb 23, 2021
@colmugx
Copy link
Member

colmugx commented Feb 23, 2021

#81 会造成部分 IME 出现异常,如果回退这个改动会让这部分 IME 用户无法正常使用

需要提供一个能避免这个问题的方案

@netlify
Copy link

netlify bot commented Feb 23, 2021

Deploy preview for v-editor ready!

Built with commit c04ba35

https://deploy-preview-128--v-editor.netlify.app

@chenEdgar
Copy link
Contributor Author

@colmugx #81 这个输入提示,目前还没有解决思路。

�待我熟悉下ckeditor渲染引擎相关的代码。

考虑原有针对 #81 的修复带来了新的问题,建议新开issue处理 #81

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants