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

Avatar component adds an alternative avatar prop #702

Closed
ZerroRt opened this issue Jul 28, 2021 · 2 comments · Fixed by #1632
Closed

Avatar component adds an alternative avatar prop #702

ZerroRt opened this issue Jul 28, 2021 · 2 comments · Fixed by #1632
Assignees
Labels
feature request New feature or request priority: low

Comments

@ZerroRt
Copy link

ZerroRt commented Jul 28, 2021

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

Automatically load alternative avatars when the src of the Avatar component loads error

Expected API (期望的 API)

The component adds a'default' attribute, or the src attribute supports passing in a string array, the first item of the array is loaded by default, and the next item of the array is automatically loaded when the loading error occurs

@github-actions github-actions bot added the feature request New feature or request label Jul 28, 2021
@XieZongChen XieZongChen self-assigned this Jul 28, 2021
@Sepush
Copy link
Collaborator

Sepush commented Nov 11, 2021

@amadeus711 any update about this?
I'm glad to work on it and give a PR.

@07akioni
Copy link
Collaborator

07akioni commented Nov 11, 2021

@amadeus711 any update about this? I'm glad to work on it and give a PR.

You can create a general image inside _internal folder and use it in avatar. We can finally move it to vueuc later.

The image component should suppport some fallback options.

Maybe a fallbackImageUrl prop and a fallback slot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request priority: low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants