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

fix(image): fix image Add the dom of the preview picture to the body #1577

Merged
merged 3 commits into from Apr 26, 2021

Conversation

hearsay316
Copy link
Contributor

Please make sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow Element's contributing guide English | (中文 | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer to relative issues for your PR.
    当image 在Carousel组件里用,fixed定位受到transform影响不能再视口定位的bug

@element-bot
Copy link
Member

@Ryan2128 Ryan2128 requested a review from SunLn March 5, 2021 14:40
@jw-foss
Copy link
Member

jw-foss commented Mar 9, 2021

Would you mind adding the added API to the documentation?
image

@hearsay316
Copy link
Contributor Author

adding the added API to the documentation

@@ -173,9 +173,10 @@ Besides the native features of img, support lazy load, custom placeholder and lo
| initial-index | The initial preview image index, less than or equal to the length of `url-list` | number | int | 0 |
| infinite | Whether preview is infinite | boolean | true / false | true |
| hide-on-click-modal | Whether user can emit close event when clicking backdrop | boolean | true / false | false |
| append-to-body | whether to append image itself to body. A nested parent element attribute transform should have this attribute set to `true` | boolean | — | false |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please copy paste this attribute description to fr es jp as well. ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

description to fr es jp

…,added documentation, description to fr es jp
@Ryan2128 Ryan2128 requested a review from jw-foss March 21, 2021 06:28
@0song 0song merged commit a87c8c3 into element-plus:dev Apr 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants