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

优化交互 #1275

Closed
wants to merge 1 commit into from
Closed

优化交互 #1275

wants to merge 1 commit into from

Conversation

ljporljp
Copy link

#1273 这是我目前为了解决这个问题的pr,想了下,做搜索的话,交互上又复杂了,其实只要加一个额外的锁定按钮,默认锁定只读即可

@qishibo
Copy link
Owner

qishibo commented Oct 25, 2024

试了一下效果还可以,不过目前还不确定像你这样误操作是不是普遍情景,加锁之后会不会对于正常重命名需求的人来说增加了操作步骤,先观望一下。
PS:不过我觉得重命名操作貌似确实也不多

@qishibo
Copy link
Owner

qishibo commented Oct 25, 2024

有俩细节问题
1、长字符的名字溢出
2、对勾那里禁止时按理说不执行操作

@ljporljp
Copy link
Author

有俩细节问题 1、长字符的名字溢出 2、对勾那里禁止时按理说不执行操作

1、长字符名字溢出是指翻译文本么?2、禁止不执行操作是指?

@qishibo
Copy link
Owner

qishibo commented Oct 25, 2024

1、就是你输入100+个字母就能看到了
2、貌似多虑了,只要不更改就不重命名弹框,我测的是打开开关更改文字,再锁定开关,这时候点击对勾还会弹重命名确认框

@ljporljp
Copy link
Author

1、就是你输入100+个字母就能看到了 2、貌似多虑了,只要不更改就不重命名弹框,我测的是打开开关更改文字,再锁定开关,这时候点击对勾还会弹重命名确认框

溢出我看到了,你是说吧对勾的位置给占了对吧。。。

@ljporljp
Copy link
Author

有什么建议么,对样式确实不太熟=。=

@qishibo
Copy link
Owner

qishibo commented Oct 25, 2024

问题不大,如果合到主分支我会改的

@ljporljp
Copy link
Author

问题不大,如果合到主分支我会改的

感谢。这个样式在哪里改我还没找到。。。。。顺带反馈下,后面的TTL的输入框,也会溢出X按钮。

@qishibo
Copy link
Owner

qishibo commented Oct 25, 2024

输入框得加padding,TTL的也有啊?哈哈等我一起改吧

@qishibo qishibo closed this in 569cf97 Nov 28, 2024
@qishibo
Copy link
Owner

qishibo commented Nov 28, 2024

想了想,还是没必要更改界面,input那里有点复杂了,索性加了个严格确认,防止误操作,和flush db时的逻辑一致
image

@ljporljp
Copy link
Author

ljporljp commented Dec 2, 2024

想了想,还是没必要更改界面,input那里有点复杂了,索性加了个严格确认,防止误操作,和flush db时的逻辑一致 image

虽然感觉这个确认框要求输入y的交互有点啰嗦,颇有种进行核试验的味道(毕竟危险等级不能核flush db相提并论),=。=但是也算是能够解决误操作的问题。

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

Successfully merging this pull request may close these issues.

2 participants