Merge pull request #13525 from petemill/fix/13524

Fix regression for default favicon color on tab hover
brave · Mar 20, 2018 · aa9db3c · aa9db3c
1 parent f0cbdf9
commit aa9db3c
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/app/renderer/components/tabs/tab.js b/app/renderer/components/tabs/tab.js
@@ -185,7 +185,7 @@ class Tab extends React.Component {
  },
  getSetting(settings.TAB_PREVIEW_TIMING))
  // fancy radial gradient mouse tracker
- if (this.elementRef) {
+ if (this.elementRef && !this.props.isActive) {
  // only update position once per render frame
  if (!this.nextFrameSetTabMouseX) {
  var x = e.pageX - this.tabOffsetLeft
@@ -566,18 +566,20 @@ const styles = StyleSheet.create({
 
  tabArea_private_active: {
  '--tab-background': theme.tab.active.private.background,
- '--tab-color': theme.tab.active.private.color,
  '--tab-background-hover': theme.tab.active.private.background,
+ '--tab-color': theme.tab.active.private.color,
  '--tab-color-hover': theme.tab.active.private.color,
- '--tab-default-icon-color': theme.tab.active.private.defaultFaviconColor
+ '--tab-default-icon-color': theme.tab.active.private.defaultFaviconColor,
+ '--tab-default-icon-color-hover': theme.tab.active.private.defaultFaviconColor
  },
 
  tabArea_themed: {
- '--tab-color': `var(--theme-color-fg)`,
  '--tab-background': `var(--theme-color-bg)`,
  '--tab-background-hover': 'var(--theme-color-bg)',
+ '--tab-color': `var(--theme-color-fg)`,
  '--tab-color-hover': 'var(--theme-color-fg)',
- '--tab-default-icon-color': 'var(--theme-color-default-icon)'
+ '--tab-default-icon-color': 'var(--theme-color-default-icon)',
+ '--tab-default-icon-color-hover': 'var(--theme-color-default-icon)'
  },
 
  tabArea__tab: {