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

feature: 统一使用svg图标,页面加载更快 #254

Closed
wants to merge 1 commit into from

Conversation

dinphy
Copy link
Contributor

@dinphy dinphy commented Jan 12, 2025

这个 PR 做了什么?

统一使用svg图标,页面加载更快

@xuewenG
Copy link
Collaborator

xuewenG commented Jan 22, 2025

这些svg和之前的图标是一样的吗?是怎么获取到的?

@dinphy
Copy link
Contributor Author

dinphy commented Jan 23, 2025

这些svg和之前的图标是一样的吗?是怎么获取到的?

一模一样的,从https://icon-sets.iconify.design/获取的

@dinphy
Copy link
Contributor Author

dinphy commented Jan 23, 2025

想了想,这样的处理方式不太适合,决定关掉这个PR

@dinphy dinphy closed this Jan 23, 2025
@xuewenG
Copy link
Collaborator

xuewenG commented Jan 23, 2025

@dinphy 可以参考下这个方法 #259

@xuewenG xuewenG added the feature New feature label Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants