Skip to content
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

建议:图标支持图片外链显示 #14

Closed
lmm214 opened this issue Feb 18, 2022 · 9 comments
Closed

建议:图标支持图片外链显示 #14

lmm214 opened this issue Feb 18, 2022 · 9 comments
Labels
加入发布火车 🚄 Meet Next Release 好点子💡 good suggestion

Comments

@lmm214
Copy link

lmm214 commented Feb 18, 2022

或者默认空则使用

http://favicon.yandex.net/favicon/google.com/

的 favicon 接口

@lmm214
Copy link
Author

lmm214 commented Feb 18, 2022

/resources/mdi-cheat-sheets/ 图标默认的这个路径有点长……

@soulteary soulteary added the 好点子💡 good suggestion label Feb 18, 2022
@soulteary
Copy link
Owner

感谢反馈 @lmm214 ,依次来回复这两个问题:

第一个问题,是能否使用外链图片,好像确实是应该支持一下,之前只考虑了性能表现、内容安全(CSP),以及风格统一。
第二个问题,我在做在线编辑功能,这个地址确实是又丑又长,我想想怎么干掉它。

第一个问题中的功能需要晚一些,代码调整和待处理的细碎事项蛮多的,第二个问题,应该下一个小版本能解决。

@soulteary soulteary added the 加入发布火车 🚄 Meet Next Release label Feb 18, 2022
@lmm214
Copy link
Author

lmm214 commented Feb 18, 2022

嗯嗯,已切换为 flare ,flame 的前端编辑虽然方便,但 node 的实时渲染还是会在打开一刹那闪烁。

@eallion
Copy link

eallion commented Feb 18, 2022

https://www.google.cn/s2/favicons?domain_url=example.com

Edit: 这个 API 不能使用,返回的图标 URL 在大陆不能访问。

@eallion
Copy link

eallion commented Feb 18, 2022

测速:
https://ping.chinaz.com/www.google.cn

@soulteary
Copy link
Owner

考虑隐私优先,或许这个功能可以内置,看了下实现成本不高。

不过为了实现更简单,也可以先选择使用 yandex、google.cn 来获取。

@soulteary
Copy link
Owner

v0.3.1 版本开始支持外链,和对没有填写图标的项目,自动获取站点 favicon。

如果需要自动获取,需要在后台打开下面的配置,选择自动获取。 @lmm214 @eallion

image

@eallion
Copy link

eallion commented Mar 10, 2022

Yandex API 的问题在于,如果一个网站本身没有设置 favicon.ico 则返回空白,Google API 却会返回一个地球图标。对网站样式设计来说 Google API 更优。

@soulteary
Copy link
Owner

Yandex API 的问题在于,如果一个网站本身没有设置 favicon.ico 则返回空白,Google API 却会返回一个地球图标。对网站样式设计来说 Google API 更优。

是的,或许直接开放存储接口,让用户自己来玩更好,比如用户能自己定义图片、favicon(减少外链还能进一步带来性能提升)。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
加入发布火车 🚄 Meet Next Release 好点子💡 good suggestion
Projects
None yet
Development

No branches or pull requests

3 participants