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

feat: 使用qrcode-svg替换qrcode #1565

Closed
wants to merge 2 commits into from

Conversation

memorydream
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Apr 26, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @qier222 on Vercel.

@qier222 first needs to authorize it.

@memorydream memorydream requested a review from pan93412 April 26, 2022 05:27
<div
v-show="qrCodeSvg"
class="qr-code-container"
v-html="qrCodeSvg"
Copy link
Collaborator

Choose a reason for hiding this comment

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

盡量避免使用 v-html

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

这个包没办法,输出就是一个完整的svg标签

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

用了一个比较脏的做法避免 v-html,请review

@pan93412
Copy link
Collaborator

🌚 還是 v-html 好了⋯⋯

@memorydream
Copy link
Collaborator Author

🌚 還是 v-html 好了⋯⋯

把这个包fork一份改一下?我想在v2里也用这个,v2里有主题色的修改,每次重新生成新的svg感觉不太合适

@memorydream
Copy link
Collaborator Author

现在我测试是没有问题了,但是这个库有一个潜在的问题,它的样式是写在svg的style

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

Successfully merging this pull request may close these issues.

2 participants