Replace favicon.ico in favor of png in src/assets
- Linguagem predominante
- TypeScript
- Estrelas
- 27k
- Forks
- 11.8k
- Merge médio
- 14h 23min
- PRs com merge (30d)
- 162
Descrição
### 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.
Guia de contribuição
Direção de pesquisa
Comece rastreando os arquivos de schematic que criam o favicon e as entradas relacionadas em angular.json; em seguida, inspecione src/assets em busca do ícone atual. Confirme quais arquivos modernos baseados em PNG ou em manifest a aplicação gerada deve incluir, remova as referências obsoletas a favicon.ico e atualize os testes de schematic relevantes se eles abrangerem os assets gerados.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- angular
- Domínio
- build-system, frontend
- Tipo de issue
- Funcionalidade
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100