Skip to content

🐛 创建用户修改名称以及id时光标跳转#125

Merged
A-kirami merged 1 commit into
A-kirami:mainfrom
roiiiu:dev
Aug 15, 2023
Merged

🐛 创建用户修改名称以及id时光标跳转#125
A-kirami merged 1 commit into
A-kirami:mainfrom
roiiiu:dev

Conversation

@roiiiu

@roiiiu roiiiu commented Aug 15, 2023

Copy link
Copy Markdown
Contributor

这个 PR 带来了什么样的更改?

  • 错误修复
  • 新功能
  • 文档/注释
  • 代码格式
  • 代码重构
  • 测试用例
  • 性能优化
  • 外观样式
  • 项目构建
  • 依赖环境
  • 持续集成/部署
  • 其他,请描述:

这个 PR 是否存在破坏性变更?

  • 是的,并已在 issue #___ 号中获得批准
  • 没有

描述

创建用户时无法正常更改用户名,每次输入光标都会跳转到字首

动机和背景

其他信息

不是很懂为什么这个form要用contenteditable的div来做,虽然看到了屎山标记🤔,如果你同意我或许可以重写一下这部分

检查工作

  • 我对我的代码进行了注释,特别是在难以理解的部分
  • 我的更改需要更新文档,并且已对文档进行了相应的更改
  • 我添加了测试并且已经在本地通过,以证明我的修复补丁或新功能有效
  • 我已检查并确保更改没有与其他打开的 Pull Requests 重复

@A-kirami A-kirami added the bug Something isn't working label Aug 15, 2023
@A-kirami

Copy link
Copy Markdown
Owner

可以进行重写,不过重写的话最好可以将 ant-design-vue 摘出去

@A-kirami A-kirami changed the title fix: 创建用户修改名称以及id时光标跳转 🐛 创建用户修改名称以及id时光标跳转 Aug 15, 2023
@A-kirami
A-kirami merged commit 8635dad into A-kirami:main Aug 15, 2023
@roiiiu

roiiiu commented Aug 17, 2023

Copy link
Copy Markdown
Contributor Author
image 项目的eslint某些设置导致代码格式崩溃🤔,能检查一下吗

@A-kirami

Copy link
Copy Markdown
Owner

image

我这边是正常的,你看看红线报错是什么

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

Labels

bug Something isn't working

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants