Skip to content

Commit

Permalink
highlight favicon on dark colored background
Browse files Browse the repository at this point in the history
- Auditors: @bradleyrichter
- Fix brave#7635
  • Loading branch information
cezaraugusto authored and NejcZdovc committed Apr 18, 2017
1 parent 997b106 commit 9dfec91
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion app/renderer/components/styles/global.js
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,8 @@ const globalStyles = {
alphaWhite: 'rgba(255,255,255,0.8)'
},
filter: {
makeWhite: 'brightness(0) invert(1)'
makeWhite: 'brightness(0) invert(1)',
whiteShadow: 'drop-shadow(0px 0px 2px rgb(255, 255, 255))'
},
radius: {
borderRadius: '4px',
Expand Down
3 changes: 2 additions & 1 deletion app/renderer/components/tabs/content/favIcon.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,8 @@ class Favicon extends ImmutableComponent {
render () {
const iconStyles = StyleSheet.create({
favicon: {
backgroundImage: `url(${this.favicon})`
backgroundImage: `url(${this.favicon})`,
filter: getTabIconColor(this.props) === 'white' ? globalStyles.filter.whiteShadow : 'none'
},
loadingIconColor: {
// Don't change icon color unless when it should be white
Expand Down

0 comments on commit 9dfec91

Please sign in to comment.