-
Notifications
You must be signed in to change notification settings - Fork 486
refactor: load the logo as an inline svg #664
refactor: load the logo as an inline svg #664
Conversation
@ruibaby is attempting to deploy a commit to the halo-dev Team on Vercel. A member of the Team first needs to authorize it. |
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.
/lgtm
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
1 similar comment
/lgtm |
#### What type of PR is this? /kind improvement /milestone 2.0 #### What this PR does / why we need it: 以 inline svg 的形式加载 Logo,解决因为加载 svg 文件导致的页面抖动问题。 #### Special notes for your reviewer: /cc @halo-dev/sig-halo-console 测试方式:检查登录页面、初始化页面、侧边菜单顶部的 Logo 是否加载正常。 #### Does this PR introduce a user-facing change? ```release-note 以 inline svg 的形式加载 Logo,解决因为加载 svg 文件导致的页面抖动问题。 ```
What type of PR is this?
/kind improvement
/milestone 2.0
What this PR does / why we need it:
以 inline svg 的形式加载 Logo,解决因为加载 svg 文件导致的页面抖动问题。
Special notes for your reviewer:
/cc @halo-dev/sig-halo-console
测试方式:检查登录页面、初始化页面、侧边菜单顶部的 Logo 是否加载正常。
Does this PR introduce a user-facing change?