Skip to content

Commit

Permalink
fix unit test by setting empty string to href whenever the favicon is…
Browse files Browse the repository at this point in the history
… not of type string (mattermost#1697)
  • Loading branch information
saturninoabril authored and jespino committed Sep 17, 2018
1 parent ff4416e commit 1a516f6
Show file tree
Hide file tree
Showing 3 changed files with 3,672 additions and 2 deletions.
4 changes: 2 additions & 2 deletions components/sidebar/sidebar.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -205,9 +205,9 @@ export default class Sidebar extends React.PureComponent {
link.rel = 'shortcut icon';
link.id = 'favicon';
if (this.badgesActive) {
link.href = redFavicon;
link.href = typeof redFavicon === 'string' ? redFavicon : '';
} else {
link.href = favicon;
link.href = typeof favicon === 'string' ? favicon : '';
}
var head = document.getElementsByTagName('head')[0];
var oldLink = document.getElementById('favicon');
Expand Down
Loading

0 comments on commit 1a516f6

Please sign in to comment.