-
Notifications
You must be signed in to change notification settings - Fork 14
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: add ai-assistant link & add home button & add a qrcode #68
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
为本功能引入 element-plus 似乎有些没必要,考虑到它会增加 bundle size
因为我们有了 unpluin-icons,所以没有必要使用 @element-plus/icons-vue
我们可以考虑把版权声明页面底部
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from a user perspective.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tweaked some design details of AI assistant
- Moved to the right-bottom corner.
- Memorize the state of if show AI assistant
This looks great |
本次功能新增说明:
1、在网站全局添加了 AI小助手图标,及link入口 #62
2、所使用的图片,署名了出处及协议内容
3、当产生视觉遮挡时,可点击close按钮,缩小图标显示
4、功能:网站左上角 新增home按钮 #69
5、功能:添加微信公众号二维码 #30