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

[Upload] There is an area of blank below the <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures #24455

Closed
1 task done
HeskeyBaozi opened this issue May 25, 2020 · 8 comments · Fixed by #26367
Labels
🐛 Bug Ant Design Team had proved that this is a bug. help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request. Inactive

Comments

@HeskeyBaozi
Copy link
Contributor

HeskeyBaozi commented May 25, 2020

  • I have searched the issues of this repository and believe that this is not a duplicate.

在表单vertical布局下上传组件(照片墙模式)添加多张图片时下方会出现很大一片空白

Reproduction link

Edit on CodeSandbox

Steps to reproduce

添加三张以上的图片,表单布局vertical,上传组件是照片墙模式

What is expected?

照片墙下没有很大一片空白

What is actually happening?

照片墙下有很大一片空白

Environment Info
antd 4.2.5
React 16.13.1
System macOS
Browser Chrome 81.0.4044.138

怀疑是换行样式有问题

@HeskeyBaozi
Copy link
Contributor Author

HeskeyBaozi commented May 25, 2020

复现操作录制视频 油管地址 https://youtu.be/Ya5BD3gLOKs

@xrkffgg
Copy link
Member

xrkffgg commented May 26, 2020

https://ant.design/components/upload-cn/#components-upload-demo-picture-card

测试了下这个 还好,没有出现你视频的情况。估计和你 Form 的定义有关吧。

@xrkffgg
Copy link
Member

xrkffgg commented May 26, 2020

Sandboxes 发下就行,还整油管 🤣

@xrkffgg
Copy link
Member

xrkffgg commented May 26, 2020

  • 用你的例子 测不出

2020-05-26 14 42 08

@HeskeyBaozi
Copy link
Contributor Author

HeskeyBaozi commented May 27, 2020

  • 用你的例子 测不出

2020-05-26 14 42 08

你可以试试传三张图片后,把中间的分栏线左拉使得照片墙的两行变成一行试试:)

@HeskeyBaozi
Copy link
Contributor Author

drag

@xrkffgg
Copy link
Member

xrkffgg commented May 27, 2020

😂😂😂 这不是人为的原因么

@afc163 afc163 added 🐛 Bug Ant Design Team had proved that this is a bug. help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request. labels May 27, 2020
@ant-design-bot
Copy link
Contributor

Hello @HeskeyBaozi. We totally like your proposal/feedback, welcome to send us a Pull Request for it. Please send your Pull Request to proper branch (feature branch for the new feature, master for bugfix and other changes), fill the Pull Request Template here, provide changelog/TypeScript/documentation/test cases if needed and make sure CI passed, we will review it soon. Appreciate it advance and we are looking forward to your contribution!

你好 @HeskeyBaozi, 我们完全同意你的提议/反馈,欢迎直接在此仓库 创建一个 Pull Request 来解决这个问题。请将 Pull Request 发到正确的分支(新特性发到 feature 分支,其他发到 master 分支),务必填写 Pull Request 内的预设模板,提供改动所需相应的 changelog、TypeScript 定义、测试用例、文档等,并确保 CI 通过,我们会尽快进行 Review,提前感谢和期待您的贡献!

giphy

@HeskeyBaozi HeskeyBaozi changed the title [Uploads] 在表单vertical布局下上传组件(照片墙模式)添加多张图片时下方会出现很大一片空白 [Uploads] There is an area of blank below the component <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures May 28, 2020
@HeskeyBaozi HeskeyBaozi changed the title [Uploads] There is an area of blank below the component <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures [Upload] There is an area of blank below the component <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures May 28, 2020
@HeskeyBaozi HeskeyBaozi changed the title [Upload] There is an area of blank below the component <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures [Upload] There is an area of blank below the <Upload listType="picture-card" /> in the <Form layout="vertical" /> when upload some pictures May 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 Bug Ant Design Team had proved that this is a bug. help wanted The suggestion or request has been accepted, we need you to help us by sending a pull request. Inactive
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants