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: fast delete current message #495

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Conversation

huangjinshe
Copy link
Contributor

单击头像后快速删除当前消息.

Copy link
Member

@BobDu BobDu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个删除的有点太猝不及防了...
至少应该在鼠标 hover 在头像上时有一个删除提示。让用户知道点下去会被快速删除。
而且对这种很容易误操作的功能。应该要有功能开关 让用户明确选择开启这种快捷删除方式。

@huangjinshe
Copy link
Contributor Author

huangjinshe commented Mar 22, 2024

容易误操作吗? 我用了那么久, 从没点过头像. 所以来了一个快准狠. 本想加个简单的功能, 你这么一说有点难为我这个非Vue科班.

@BobDu
Copy link
Member

BobDu commented Mar 22, 2024

确实当用户比较多的时候。每动一个小功能都会有很多顾虑。重口难调。
如果真这么快准狠了。那肯定会有人提issue抱怨为什么点一下头像就把数据删了

要不要再琢磨琢磨怎么加个功能开关。正好学习学习vue

@huangjinshe
Copy link
Contributor Author

嗯嗯, 我看下方不方便在设置--总览 最下面加个Checkbox.(下面几个设置刚好和会话有关)

@BobDu
Copy link
Member

BobDu commented Mar 22, 2024

嗯。有个开关能默认不开启就问题不大了。
还有就是我感受了一下。双击删除是不是比单击删除好一些了?

@xiaosaaaa
Copy link

双击好

@huangjinshe huangjinshe changed the title feat: add fast delete current message by click avatar feat: fast delete current message Mar 23, 2024
@huangjinshe
Copy link
Contributor Author

huangjinshe commented Mar 23, 2024

已改.

test11

test2

@BobDu BobDu merged commit 5899b04 into chatgpt-web-dev:main Mar 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants