Remove favicons and links in header.php

[crftwrk]

Closes #419
PR in child bootscore/bootscore-child#34

This PR removes the favicons and links to them in header.php

Why?

• It's against the WordPress codex. These links were one (now the last) reason why we couldn't publish to the WP theme repository.
• Buggy with Android Chrome and Windows 8/10.
• Users do not take this step serious enough. Most developers works on Windows with Chrome, and didn’t check on any other device. I'm an Apple user and I see beautiful designed sites but viewing them in Safari shows still the blue b :-/

Think we should give this back to the Customizer and mention in the docs that the 90kb plugin https://wordpress.org/plugins/favicon-by-realfavicongenerator/ leads to the best results.

When merged, we can theoretically ask for a theme review in the WP repository, question is if we want to go this way.

@justinkruit if you agree, please merge.

[justinkruit]

I'm an Apple user and I see beautiful designed sites but viewing them in Safari shows still the blue b :-/

That's actually quite funny 😂

When merged, we can theoretically ask for a theme review in the WP repository, question is if we want to go this way.

Personally, not the biggest fan. I think this theme relies on a developer making changes to make it fit their needs. For me the WP repository is more suited for non-developers.

[crftwrk]

For me the WP repository is more suited for non-developers.

Agree