tinycon icon indicating copy to clipboard operation
tinycon copied to clipboard

Regexp/Delete too aggressive?

Open justinaray opened this issue 8 years ago • 1 comments

Looking at the source and my head tag, it looks like all of my supplemental icon links are being removed. Specifically, this includes the apple-touch-icon[-precomposed].

Is this necessary to ensure the tinycon dynamic icon is applied on all platforms? Would it be possible to whitelist in some rel types that shouldn't be removed?

tinycon is 💯 btw!

justinaray avatar Dec 06 '16 16:12 justinaray

Using document.querySelectorAll('head link[rel~="icon"]') could also be an approach instead of whitelisting. It would select just link tags that contain an icon string in the rel attribute.

Presumably, it would be acceptable to remove all of these tags as they are favicon candidates.

justinaray avatar Dec 06 '16 17:12 justinaray