Merged
Conversation
Member
|
麻烦确定一下正确性,而且感觉不用这么多,选择常见的即可 对新增的 9 个图标服务逐个用 curl 实测(对比 github.com / youtube.com 返回 MD5 是否不同)。 ❌ 严重问题:Baidu 服务完全无效
注释里把它吹成"国内最佳 fallback 之一"属于严重误导。建议直接移除该选项。 ✅ 其他 8 个服务实测均正常
💡 精简建议9 个新增服务能力高度重复,只是不同厂牌的小站点。个人建议裁剪成这 4 个:
如果想再加一个小众备选,建议留 favicone 或 icon-horse 任选其一(二者定位完全一样)。 建议删除的 5 个
4 个选项足以覆盖所有用户场景,UI 也更清爽。 |
Member
|
也太多啦,可以按照建议砍一下 |
根据 PR 评论反馈,移除画质过差、同质化、不稳定的 6 个服务 (favicone, favicon.im, vemetric, yandex, allesedv, manifest.im), 只保留 scriptcat、google、duckduckgo、icon-horse、local 5 个核心选项, 让 UI 更清爽。同时清理 zh-CN/zh-TW 残留的 baidu 翻译键。
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.
Checklist / 检查清单
Description / 描述
@CodFrm 除了 DuckDuckGo, Yandex , 其他新增的应该都可以在大陆使用
Screenshots / 截图