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(chat): add Chat component #518

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

mortalYoung
Copy link
Collaborator

变更类型

请选择以下选项以描述 PR 的类型:

  • Bug 修复(修复现有问题)
  • 新功能(添加了一个功能)
  • 代码优化(性能改进、代码重构)
  • 文档更新
  • 单测新增或修改
  • 其他(请说明):

相关问题

变更内容

  • 新增 Chat 组件

详细描述

对应 Previewer

Previewer

@mumiao mumiao self-requested a review November 27, 2024 08:23
@liuxy0551
Copy link
Collaborator

  1. 点击顶部的 link1 link2 进行切换,会打断输出并无法恢复
  2. 停止回答的 icon 和文字不水平
image

@liuxy0551
Copy link
Collaborator

  1. 第三个按钮点击后颜色变化,但需要点击其他地方才能恢复原样
    2024-12-02 14 42 46

@liuxy0551
Copy link
Collaborator

liuxy0551 commented Dec 2, 2024

  1. value 参数重复
  2. 参数的默认值都没有写,但看代码有些算是有默认值的
    image

@liuxy0551
Copy link
Collaborator

  1. 标题错误
image

@mortalYoung mortalYoung requested a review from liuxy0551 December 3, 2024 07:07
@mortalYoung mortalYoung changed the title WIP: feat(chat): add Chat component feat(chat): add Chat component Jan 6, 2025
@mortalYoung mortalYoung requested a review from Cythia828 January 6, 2025 02:27
@LuckyFBB
Copy link
Collaborator

image

default 的 icon 和 文本没有对齐

const copyInfo = useMemo<{ disabled: boolean; options: CopyOptions }>(() => {
if (typeof copy === 'boolean') {
return {
disabled: copy,
Copy link
Collaborator

Choose a reason for hiding this comment

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

如果 copy 是 true,那么会设置为 disabled,renderCopyIcon 中会直接返回 null


## API

### Params
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个params 不太对头吧

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.

4 participants