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

支持 dir 设置 #1197

Closed
HerbertHe opened this issue Mar 16, 2022 · 17 comments
Closed

支持 dir 设置 #1197

HerbertHe opened this issue Mar 16, 2022 · 17 comments
Assignees
Milestone

Comments

@HerbertHe
Copy link
Contributor

编辑模式

  • wysiwyg 所见即所得模式
  • ir 即时渲染模式
  • sv 分屏编辑预览模式

描述问题

默认情况下,键入 markdown 都遵循 LTR 的使用习惯,但是针对于 RTL 的语言没有特别的优化。会导致 LTR 和 RTL 文本混排的时候 markdown 的 加粗、斜体 等feat出现布局位置错误的情况,而 RTL 的布局会考虑修复这种缺陷

期待的结果

截屏或录像

版本信息

  • 版本:latest
  • 操作系统:Win10
  • 浏览器:Edge

其他信息

@HerbertHe HerbertHe mentioned this issue Mar 16, 2022
@Vanessa219 Vanessa219 assigned Vanessa219 and HerbertHe and unassigned Vanessa219 Mar 22, 2022
@Vanessa219 Vanessa219 added this to the 3.8 milestone Mar 22, 2022
@Vanessa219 Vanessa219 changed the title RTL 游标定位错误 支持 dir 设置 Mar 22, 2022
Vanessa219 added a commit that referenced this issue Mar 22, 2022
@Vanessa219
Copy link
Owner

感觉问题有点多呀,还需要再改改。
image

@Vanessa219 Vanessa219 reopened this Mar 22, 2022
Vanessa219 added a commit that referenced this issue Mar 22, 2022
Vanessa219 added a commit that referenced this issue Mar 22, 2022
Vanessa219 added a commit that referenced this issue Mar 22, 2022
Vanessa219 added a commit that referenced this issue Mar 22, 2022
@HerbertHe
Copy link
Contributor Author

感觉问题有点多呀,还需要再改改。 image

主要问题是,如果开启RTL需要将语言提示也改掉。

@HerbertHe
Copy link
Contributor Author

@Vanessa219 我仔细看了一下,能修复样式的问题,包括工具栏的样式也改成 RTL。那个 label 似乎是通过 aria-label 实现的?这似乎是 CSS 样式缺陷导致的emmm

@HerbertHe
Copy link
Contributor Author

针对子工具栏的问题,增加 DOM 和修改 CSS 样式可以解决。

@HerbertHe
Copy link
Contributor Author

我感觉迁移到 Less,并且重构样式表似乎是必要i的,可以借助三方工具来进一步减小样式表的体积。

@Vanessa219
Copy link
Owner

如果是打包工具的话,可以看一下具体减少的大小和使用成本。减少不大的话可以先不引入。

@Vanessa219
Copy link
Owner

#1200

@HerbertHe
Copy link
Contributor Author

如果是打包工具的话,可以看一下具体减少的大小和使用成本。减少不大的话可以先不引入。

还有个问题是,Windows 端的 Sass 环境配置始终是个大问题 emmm,尤其是本地已经有 Python3

Vanessa219 added a commit that referenced this issue Mar 28, 2022
@Vanessa219
Copy link
Owner

如果是打包工具的话,可以看一下具体减少的大小和使用成本。减少不大的话可以先不引入。

还有个问题是,Windows 端的 Sass 环境配置始终是个大问题 emmm,尤其是本地已经有 Python3

PR 合并了,可以重构 sass -> less 了

@HerbertHe
Copy link
Contributor Author

PR 合并了,可以重构 sass -> less 了

我看时间重构一下,可能部分样式得重写了

@Vanessa219
Copy link
Owner

嗯嗯,重构完多测测。

pnpm 用下来感觉有些依赖要手动添加,npm 就不会,太奇怪了。

@HerbertHe
Copy link
Contributor Author

嗯嗯,重构完多测测。

pnpm 用下来感觉有些依赖要手动添加,npm 就不会,太奇怪了。

依赖的话npm和pnpm都是一样的,可能之前注释掉了sass的包,就得手动添加了。我打算更新测试用例了,这样也能及时发现问题。。。emmm不过得过段时间了,摔着腰住院了😂

@Vanessa219
Copy link
Owner

多休息,祝早日康复。

vditor 没这个问题,我是把思源换成 pnpm,打安装包的时候就残了。

@HerbertHe
Copy link
Contributor Author

哈哈哈,俺也不知道思源源码,没法发现问题哈哈哈哈🙈不过我感觉vditor还是建一个组织仓库集中维护比较好一点

@Vanessa219
Copy link
Owner

就是 electron-builder 的时候有几个项目内没有用到的依赖会导致无法启动。

目前没有太多的精力,后面有空可以迁移一下。

@HerbertHe
Copy link
Contributor Author

目前确实还没遇到过这个问题emmm

@Vanessa219
Copy link
Owner

先关闭了,后期有精力再折腾。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants