💄 style(ui): Confirmation when adding a new topic
#3030
Conversation
|
@RubuJam is attempting to deploy a commit to the LobeHub Pro Team on Vercel. A member of the Team first needs to authorize it. |
|
👍 @RubuJam Thank you for raising your pull request and contributing to our Community |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3030 +/- ##
==========================================
+ Coverage 93.40% 93.87% +0.46%
==========================================
Files 380 386 +6
Lines 23477 23824 +347
Branches 1786 1765 -21
==========================================
+ Hits 21929 22364 +435
+ Misses 1548 1460 -88
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
@arvinxx Please review |
| ); | ||
| if (mobile) { | ||
| return ( | ||
| <Popconfirm |
There was a problem hiding this comment.
Mobile 上不要用 Popconfirm 吧，这个有点太小了。换成 Modal .confirm 好一些。另外逻辑写在 onClick 里，不建议再单独提一个 组件 Render 出来
There was a problem hiding this comment.
Mobile上的即将清空当前会话消息，清空后将无法找回，请确认你的操作也是用的Popconfirm，实际效果也还可以，我倾向于不换，而且现在的布局更方便用户单手操作，如果有必要换，那我就去改
这是现在的显示效果
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
在移动端点击
将当前会话保存为话题时，会进行弹窗进行二次确认，避免误触在移动设备上，话题列表中的
MoreVertical图标将会始终显示Upon selecting the option to "Save current conversation as topic" on the mobile terminal, a pop-up window will be displayed for secondary confirmation, thus preventing any potential accidental touches.
In the context of mobile devices, the "MoreVertical" icon will invariably be present within the topic list.
📝 补充信息 | Additional Information
close #2717
