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

Image.Preview 不应该用动画放大控件 #36456

Closed
afc163 opened this issue Jul 11, 2022 · 3 comments · Fixed by #38415
Closed

Image.Preview 不应该用动画放大控件 #36456

afc163 opened this issue Jul 11, 2022 · 3 comments · Fixed by #38415
Assignees
@afc163
Labels
improvement improve some features Inactive 👷🏻‍♂️ Someone working on it

Comments

@afc163
Copy link
Member

afc163 commented Jul 11, 2022 •
edited

https://ant.design/components/image-cn/#components-image-demo-preview-group-visible

应该只有图片本身走动画放大。@shaodahong
@afc163 afc163 added the improvement improve some features label Jul 11, 2022
@shaodahong
Copy link
Member

应该只有图片本身走动画放大

我看现在是放大图片本身，这里指的是？

@afc163
Copy link
Member Author

afc163 commented Jul 12, 2022

缩放按钮，转向按钮，mask，关闭按钮，现在会随图片打开也走一次放大的动画。

@shaodahong
Copy link
Member

缩放按钮，转向按钮，mask，关闭按钮，现在会随图片打开也走一次放大的动画。

嗯，这个是 Modal 组件的放大逻辑，这里可以优化下，在 modal animate 结束后再显示操作栏

@afc163 afc163 assigned afc163 and unassigned shaodahong Nov 5, 2022
This was referenced Nov 6, 2022
Merged
Merged
@afc163 afc163 mentioned this issue Nov 12, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@afc163 afc163

Labels
improvement improve some features Inactive 👷🏻‍♂️ Someone working on it
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Image preview controls should not zoom out ant-design/ant-design
2 participants
@afc163 @shaodahong