-
Notifications
You must be signed in to change notification settings - Fork 484
feat: add support for setting console's favicon #788
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
如果 favicon 能被修改,logo 不能被修改会不会有有点奇怪。 |
现在没有这样的需求,而且产品 Logo 一般也是不让改的吧。 |
是这么个道理,但是 logo 和 favicon 的配置都在这个地方,一个影响控制台一个不影响就会觉得有些怪。 |
@wan92hen 那么有什么建议吗🌚 |
|
确实是有计划的,halo-dev/halo#2582 (comment) 稍后我新建一个 issue。 |
当前还是按照这个 PR 进行。 /cc @halo-dev/sig-halo-console |
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.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang, wzrove 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 |
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
#### What type of PR is this? /kind feature #### What this PR does / why we need it: Console 支持使用系统设置中的 Favicon。 #### Which issue(s) this PR fixes: Fixes halo-dev#3010 #### Screenshots: ![2022-12-20 19 40 48](https://user-images.githubusercontent.com/21301288/208658754-d62f9d46-7f15-412f-80e2-17fd9729ca8f.gif) #### Special notes for your reviewer: 测试方式: 1. 进入系统设置。 2. 设置任意一个图片为 Favicon。 3. 观察 Console 的站标是否被修改。 #### Does this PR introduce a user-facing change? ```release-note Console 端支持设置 Favicon ```
What type of PR is this?
/kind feature
What this PR does / why we need it:
Console 支持使用系统设置中的 Favicon。
Which issue(s) this PR fixes:
Fixes halo-dev/halo#3010
Screenshots:
Special notes for your reviewer:
测试方式:
Does this PR introduce a user-facing change?