feat(dashboard): dashboard 增加 favicon 与 apple-touch-icon(裁掉 logo 留白放大图标)#312
Merged
Conversation
裁掉原 botmux logo 四周约 20% 的留白,把 bn 图标放大居中,生成 favicon.png(512²) 与 apple-touch-icon.png(180²),背景沿用原 logo 浅紫。 - index.html <head> 增加 <link rel="icon"> 与 <link rel="apple-touch-icon"> - dashboard:bundle 增加两个图标拷贝到 dist/dashboard-web - 走现有 /assets/ 静态路由,与 style.css 同机制
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
改动
给 dashboard 增加 favicon(浏览器标签页图标),并顺带补了 apple-touch-icon(移动端加到桌面的图标)。
按需求裁掉原 botmux logo 四周约 20% 的留白,把
bn图标放大居中:favicon.png(512²) 与apple-touch-icon.png(180²),背景沿用原 logo 的浅紫index.html<head>增加<link rel="icon">与<link rel="apple-touch-icon">package.json的dashboard:bundle增加两个图标到dist/dashboard-web的拷贝步骤/assets/静态路由(与style.css/app.js同机制)采用「仅图标」裁法:16/32px 标签页下也清晰。
验证
pnpm build通过;确认favicon.png/apple-touch-icon.png落到dist/dashboard-web/,shippedindex.html含两个 link 标签