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

更改favicon为本地 #13

Open
weiyidu opened this issue Apr 9, 2021 · 7 comments
Open

更改favicon为本地 #13

weiyidu opened this issue Apr 9, 2021 · 7 comments

Comments

@weiyidu
Copy link

@weiyidu weiyidu commented Apr 9, 2021

代码中的获取不了,自己在网上找了方法修改了一下。
源码 https://github.com/owen0o0/getFavicon

效果 https://www.daohangweb.com

在模版index.php中找到如下代码:
<img src="https://favicon.rss.ink/v1/<?php echo base64($link['url']); ?>" alt="HUAN" width="16" height="16" /> 修改成 <img src="/favicon/<?php echo $link['url']; ?>" alt="HUAN" width="16" height="16" />
另外也建议把一些在线的css文件都改成本。

@helloxz
Copy link
Owner

@helloxz helloxz commented Apr 9, 2021

  1. 后续会考虑css/js完全离线
  2. favicon离线我先看一下
@weiyidu
Copy link
Author

@weiyidu weiyidu commented Apr 9, 2021

  1. 后续会考虑css/js完全离线
  2. favicon离线我先看一下

操作后台时发现可以批量选择,好像是没有批量删除。

@weiyidu
Copy link
Author

@weiyidu weiyidu commented Apr 9, 2021

  1. 后续会考虑css/js完全离线
  2. favicon离线我先看一下

如果能支持多级目录就更妙了

@woodxi
Copy link

@woodxi woodxi commented Apr 25, 2021

代码中的获取不了,自己在网上找了方法修改了一下。
源码https://github.com/owen0o0/getFavicon

效果https://www.daohangweb.com

在模版index.php中找到如下代码:另外
<img src="https://favicon.rss.ink/v1/<?php echo base64($link['url']); ?>" alt="HUAN" width="16" height="16" /> 修改成 <img src="/favicon/<?php echo $link['url']; ?>" alt="HUAN" width="16" height="16" />
也建议把一些在线的css文件都改成本。

@woodxi
Copy link

@woodxi woodxi commented Apr 25, 2021

楼上的大神,你那个页面很棒,不知道能否共享源码!谢谢了

@weiyidu
Copy link
Author

@weiyidu weiyidu commented Apr 26, 2021

楼上的大神,你那个页面很棒,不知道能否共享源码!谢谢了

https://hostloc.com/thread-831766-1-1.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@helloxz @weiyidu @woodxi and others