Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Zooming value does not initialize when switching a picture in N-Image-Group Component #423

Closed
LYErin opened this issue Jul 7, 2021 · 2 comments · Fixed by #430
Closed

Zooming value does not initialize when switching a picture in N-Image-Group Component #423

LYErin opened this issue Jul 7, 2021 · 2 comments · Fixed by #430

Comments

@LYErin
Copy link
Contributor

LYErin commented Jul 7, 2021

TuSimple/naive-ui version (版本)

2.15.3

Vue version (Vue 版本)

3.1.4

Browser and its version (浏览器及其版本)

91.0.4472.124

System and its version (系统及其版本)

win10

Node version (Node 版本)

Reappearance link (重现链接)

https://codesandbox.io/s/jidld

Reappearance steps (重现步骤)

1、 zooming a picture in N-Image-Group Component
2、switch a picture
3、zoom this picture; the zooming effect is not right!

Expected results (期望的结果)

Zooming value initialized when switching a picture

Actual results (实际的结果)

The zooming effect is not right!

Remarks (补充说明)
@Fnzero Fnzero added bug Something isn't working and removed bug Something isn't working labels Jul 7, 2021
@Fnzero
Copy link
Collaborator

Fnzero commented Jul 7, 2021

we cannot reproduce it.

@LYErin
Copy link
Contributor Author

LYErin commented Jul 7, 2021 •
edited
Loading

I have resolved this problem but can not covered the test.
pr link

@07akioni 07akioni mentioned this issue Jul 7, 2021
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: n-image-group zoom scale initialied LYErin/naive-ui
2 participants
@Fnzero @LYErin