Replace favicon.ico in favor of png in src/assets
- Ngôn ngữ chính
- TypeScript
- Star
- 27k
- Fork
- 11.8k
- Merge trung bình
- 14 giờ 23 phút
- Pull request đã merge (30 ngày)
- 162
Mô tả
### Bug Report or Feature Request (mark with an `x`)
```
- [ ] bug report -> please search issues before submitting
- [x] feature request
```
### Area
```
- [ ] devkit
- [x] schematics
```
### Desired functionality
favicon.ico is outdated and obsolete relic from the 90s. Browsers for the past 2decades have supported png favicons. Additionally, while it's fairly easy now to generate a favicon.ico, it's just not needed or used anymore.
There's no reason imo to include favicon.ico, even for "compatibility". Removing the favicon.ico will also remove the entries needed in angular.json and remove the only asset stored outside of src/assets.
It should be replaced in favor of modern favicon methods. PWAs should use manifest.json
```
```
Note: svg doesn't have full browser support.
We haven't needed favicon.ico in years, but it's still widely used and implemented, even though all browsers support modern methods.
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách lần theo các tệp schematic tạo favicon và các mục liên quan trong angular.json, sau đó kiểm tra src/assets để tìm biểu tượng hiện tại. Xác nhận ứng dụng được tạo cần bao gồm những tệp hiện đại nào dựa trên PNG hoặc manifest, xóa các tham chiếu đã lỗi thời đến favicon.ico và cập nhật các bài kiểm thử schematic liên quan nếu chúng bao quát các asset được tạo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- angular
- Lĩnh vực
- build-system, frontend
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100