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

Please, please be optimistic about the problem, I can not set the rounded corner picture #427

Closed
hairyf opened this issue Jul 7, 2021 · 5 comments

Comments

@hairyf
Copy link

hairyf commented Jul 7, 2021

This function solves the problem (这个功能解决的问题)

This is the way I followed yours, the picture does not appear rounded corners

image
image
#409

Expected API (期望的 API)

#409

@Talljack
Copy link
Contributor

Talljack commented Jul 7, 2021

You can set style="border-radius:50%;"on img,like that
image

@hairyf
Copy link
Author

hairyf commented Jul 7, 2021

Yes, I know, it can be set like this

<n-image class="my-image" width="32" src="https://07akioni.oss-cn-beijing.aliyuncs.com/07akioni.jpeg" />
.my-image img {
   border-radius: 50%;
}

But I think it should be like this

<n-image style="border-radius:50%;" width="32" src="https://07akioni.oss-cn-beijing.aliyuncs.com/07akioni.jpeg" />

or

<n-image rounded width="32" src="https://07akioni.oss-cn-beijing.aliyuncs.com/07akioni.jpeg" />

Okay, maybe you think this is a bit unreasonable, this is the last time I have raised this issue.

I also know that this reason is for compatible viewing of pictures, so there are multiple roots of the component, which makes the styles unable to be inherited.

@Talljack
Copy link
Contributor

Talljack commented Jul 7, 2021

I think you can setting the border-radius of img to inherit can resolve the problem
If it is round, the requirement must be square, this will destroy the picture

@07akioni
Copy link
Collaborator

07akioni commented Jul 7, 2021

@TuiMao233 We will fix it

@07akioni
Copy link
Collaborator

07akioni commented Jul 7, 2021

Now border radius should work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants