Skip to content

Commit

Permalink
Set title instead of favicon for IE11 and Edge too.
Browse files Browse the repository at this point in the history
Both actually support canvas and the favicon link is set correctly. But for some unknown reason, they don't support setting favicons dynamically to data. Oh my.

Bug: 14189
  • Loading branch information
yunosh committed Dec 17, 2015
1 parent 15e1cc1 commit 63a021a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion imp/js/external/tinycon.js
Expand Up @@ -60,7 +60,7 @@
}());

var browser = {
ie: ua('msie'),
ie: ua('msie') || ua('trident') || ua('edge'),
chrome: ua('chrome'),
webkit: ua('chrome') || ua('safari'),
safari: ua('safari') && !ua('chrome'),
Expand Down

0 comments on commit 63a021a

Please sign in to comment.