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.

Sign up for GitHub

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

Jump to bottom

ColorPicker review issues #42333

Closed
1 task done
afc163 opened this issue May 13, 2023 · 22 comments
Closed
1 task done

ColorPicker review issues #42333

afc163 opened this issue May 13, 2023 · 22 comments
Labels
🐛 Bug Ant Design Team had proved that this is a bug.

Comments

@afc163
Copy link
Member

afc163 commented May 13, 2023 •
edited by zombieJ
Loading

图片
  • 弹出动画方向有点不对，上面这个是从中间弹开的，应该从箭头方向弹开。可以参考 Popover。

发现 https://preview-42329-ant-design.surge.sh/components/popover-cn 也丢了，不是 ColorPicker 的问题，是其他地方改挂了。
@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by li-jia-nan
Loading

图片
  • 操作点 hover 时没有出现鼠标手型。

@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by zombieJ
Loading

图片

@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by li-jia-nan
Loading

图片

@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by li-jia-nan
Loading 

import type { Color } from 'antd/es/color-picker';
  • 建议都用 es 目录。

@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by zombieJ
Loading

图片
  • 这个图案是重复的，感觉 16X16 + repeat 就够了，不需要这么大。

@afc163
Copy link
Member Author

afc163 commented May 13, 2023 •
edited by li-jia-nan
Loading

@li-jia-nan li-jia-nan added 🐛 Bug Ant Design Team had proved that this is a bug. and removed unconfirmed labels May 13, 2023
@li-jia-nan li-jia-nan mentioned this issue May 13, 2023
Merged
20 tasks
@li-jia-nan
Copy link
Member

li-jia-nan commented May 13, 2023 •
edited
Loading

  • 操作点 hover 时没有出现鼠标手型。

#42334 done~

@li-jia-nan
Copy link
Member

li-jia-nan commented May 13, 2023 •
edited
Loading

  • 描述里少 。
  • 可以在这里提醒用户是 5.*.0 版本后新增的组件。

#42334 done~

@li-jia-nan
Copy link
Member

li-jia-nan commented May 13, 2023 •
edited
Loading

  • 建议都用 es 目录。

#42334 done~

@li-jia-nan
Copy link
Member

发现 https://preview-42329-ant-design.surge.sh/components/popover-cn 也丢了，不是 ColorPicker 的问题，是其他地方改挂了。

这个需要 @zombieJ 豆酱大佬看看

@li-jia-nan
Copy link
Member

li-jia-nan commented May 13, 2023 •
edited
Loading

  • 这个图案是重复的，感觉 16X16 + repeat 就够了，不需要这么大。

这个需要 @RedJue 大佬看看，是不是需要 @MadCcc 闲夕大佬提供一个 16x16 的素材

@li-jia-nan
Copy link
Member

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

不知道我理解对不对，需要 @MadCcc @RedJue 两位大佬看看

@RedJue
Copy link
Member

RedJue commented May 14, 2023

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

不知道我理解对不对，需要 @MadCcc @RedJue 两位大佬看看

是这样的，这里变成受控了，所以没法切换 format

@RedJue
Copy link
Member

RedJue commented May 14, 2023

  • 这个图案是重复的，感觉 16X16 + repeat 就够了，不需要这么大。

这个需要 @RedJue 大佬看看，是不是需要 @MadCcc 闲夕大佬提供一个 16x16 的素材

需要的，要重新弄个16*16的

@afc163
Copy link
Member Author

afc163 commented May 14, 2023

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

demo 里不要出现让用户费解的交互，这里 format + onChange 配合吧。

@li-jia-nan
Copy link
Member

  • 这个图案是重复的，感觉 16X16 + repeat 就够了，不需要这么大。

这个需要 @RedJue 大佬看看，是不是需要 @MadCcc 闲夕大佬提供一个 16x16 的素材

需要的，要重新弄个16*16的

那这个等闲夕大佬周一上班了提供一个素材吧

@li-jia-nan
Copy link
Member

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

demo 里不要出现让用户费解的交互，这里 format + onChange 配合吧。

这样的话是不是 demo 中只需要一个就够了，不用出现三次了，@RedJue 大佬要不你来看一下？

@RedJue
Copy link
Member

RedJue commented May 14, 2023

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

demo 里不要出现让用户费解的交互，这里 format + onChange 配合吧。

嗯嗯 可以加个 onFormatChange 配合一下

@RedJue
Copy link
Member

RedJue commented May 14, 2023

这个我的理解是预期行为，因为这里本来默认非受控模式，但是 demo 中指定了 format，那么就变成了受控模式，所以切换就不起作用了吧

demo 里不要出现让用户费解的交互，这里 format + onChange 配合吧。

这样的话是不是 demo 中只需要一个就够了，不用出现三次了，@RedJue 大佬要不你来看一下？

写3个只是为了显示更清楚，比较直观展现 format 的效果，还是必要的，就把受控优化一下好了，demo 还是减少用户理解的成本，简单处理。

@jinboren
Copy link

+1

@linxianxi
Copy link
Contributor

应该支持下 #fff 这种就三个英文的色号

@RedJue RedJue mentioned this issue May 15, 2023
Merged
20 tasks
@afc163 afc163 mentioned this issue May 15, 2023
Merged
This was referenced May 15, 2023
Merged
Merged
@RedJue RedJue mentioned this issue May 15, 2023
Merged
20 tasks
@RedJue
Copy link
Member

RedJue commented May 15, 2023

应该支持下 #fff 这种就三个英文的色号

后续版本可以考虑支持

@zombieJ zombieJ closed this as completed May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 Bug Ant Design Team had proved that this is a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@afc163 @zombieJ @RedJue @linxianxi @li-jia-nan @jinboren