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(ColorPicker): add ColorPicker component #6722

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

CCherry07
Copy link
Member

首先,感谢你的贡献! 😄

新特性请提交至 feature 分支,其余可提交至 master 分支。在一个维护者审核通过后合并。请确保填写以下 pull request 的信息,谢谢!~

[English Template / 英文模板]

这个变动的性质是

  • 新特性提交
  • 日常 bug 修复
  • 站点、文档改进
  • 组件样式改进
  • TypeScript 定义更新
  • 重构
  • 代码风格优化
  • 分支合并
  • 其他改动(是关于什么的改动?)

需求背景

  1. 描述相关需求的来源。
  2. 要解决的问题。
    新增 ColorPicker组件
  3. 相关的 issue 讨论链接。

实现方案和 API(非新功能可选)

  1. 基本的解决思路和其他可选方案。
  2. 列出最终的 API 实现和用法。
  1. 涉及 UI/交互变动需要有截图或 GIF。

对用户的影响和可能的风险(非新功能可选)

  1. 这个改动对用户端是否有影响?影响的方面有哪些?
    新增特性,无影响
  2. 是否有可能隐含的 break change 和其他风险?

Changelog 描述(非新功能可选)

  1. 英文描述
  2. 中文描述(可选)

请求合并前的自查清单

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

后续计划(非新功能可选)

如果这个提交后面还有相关的其他提交和跟进信息,可以写在这里。

@wsxe9988
Copy link

大哥牛逼
ant-design/ant-design@a5fa9e2#diff-1224d5ca5233fdeba67ba2e2dc6fa3e615905253f35dc132c4b4b86f51f9eab4 可以直接对标到ant-design 5.7.0里的color-picker

@CCherry07
Copy link
Member Author

大哥牛逼 ant-design/ant-design@a5fa9e2#diff-1224d5ca5233fdeba67ba2e2dc6fa3e615905253f35dc132c4b4b86f51f9eab4 可以直接对标到ant-design 5.7.0里的color-picker

如你所愿,😴 😴😴

@wsxe9988
Copy link

大哥牛逼 ant-design/ant-design@a5fa9e2#diff-1224d5ca5233fdeba67ba2e2dc6fa3e615905253f35dc132c4b4b86f51f9eab4 可以直接对标到ant-design 5.7.0里的color-picker

如你所愿,😴 😴😴

辛苦的,ant-design更新的太快了

@tangjinzhou tangjinzhou changed the base branch from feat-v4 to main August 6, 2023 04:49
@ds3783
Copy link

ds3783 commented Sep 20, 2023

@tangjinzhou 希望尽早review ,早日merge 进 master啊

@zmcode
Copy link

zmcode commented Sep 28, 2023

感谢开源

@zmcode
Copy link

zmcode commented Sep 28, 2023

希望尽快合并, 我尝试使用这个分支的代码, 发现这个组件还没完善, 少了vc-color-picker的样式文件 还有主动修改input的颜色, 无法改变取色器的位置

@zmcode
Copy link

zmcode commented Sep 29, 2023

@CCherry07
我尝试了这个组件, 目前发现如下问题

image
useColorDrag hook内部对color的更改进行了监听, 但是颜色改变并不会执行, 导致初始化的时候无法正确定位圆点的位置

  1. ColorPickerPanel组件 内部需要渲染的元素希望提供确定按钮, 在一些表单场景中, 快速变化颜色, 会导致不断触发表单的值改变, 造成性能问题

image

@CCherry07
Copy link
Member Author

@CCherry07 我尝试了这个组件, 目前发现如下问题

image useColorDrag hook内部对color的更改进行了监听, 但是颜色改变并不会执行, 导致初始化的时候无法正确定位圆点的位置

  1. ColorPickerPanel组件 内部需要渲染的元素希望提供确定按钮, 在一些表单场景中, 快速变化颜色, 会导致不断触发表单的值改变, 造成性能问题

image

问题1:你好像没有更新到最新的版本
问题2:有一个api onChangeComplete 这个是你选择完之后才会生效

@CCherry07
Copy link
Member Author

CCherry07 commented Oct 1, 2023

希望尽快合并, 我尝试使用这个分支的代码, 发现这个组件还没完善, 少了vc-color-picker的样式文件 还有主动修改input的颜色, 无法改变取色器的位置

  1. vc-color-picker的样式并不需要,样式直接有 Color Picker 提供
  2. 动修改input的颜色 hex input 你需要输入合法的hex格式,#FFF 这种是错误的 #FFFFFF OK

@WhCollar
Copy link

Please, implement this, i need it

@wsxe9988
Copy link

@CCherry07 @tangjinzhou 啥时候合并呢

@AutismPatient
Copy link

Is there any progress on this merger

@vellengs
Copy link

还没成功合并么?

Copy link

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label May 14, 2024
@zmcode
Copy link

zmcode commented May 14, 2024 via email

@github-actions github-actions bot removed the Stale label May 15, 2024
Copy link

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

@github-actions github-actions bot added the Stale label Jul 14, 2024
@github-actions github-actions bot closed this Jul 22, 2024
@wsxe9988
Copy link

wsxe9988 commented Aug 3, 2024

@selicens @CCherry07 @tangjinzhou 这个怎么都自动关闭了,啥时候合并进去啊

@selicens selicens reopened this Aug 29, 2024
@github-actions github-actions bot removed the Stale label Aug 30, 2024
@kkfive
Copy link

kkfive commented Oct 14, 2024

mark

@ColourfulU
Copy link

为什么找不到colorpicker分支了

@zmcode
Copy link

zmcode commented Dec 9, 2024 via email

@quarksb
Copy link

quarksb commented Jan 23, 2025

什么时候合进去啊,现在有啥问题吗?

@zmcode
Copy link

zmcode commented Jan 23, 2025 via email

@IF-tiger
Copy link

IF-tiger commented Mar 3, 2025

我也想问下颜色选择器什么时候才能合进下一个主版本

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.