Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code
Issues
All Issues

Unified Diff: mobile/android/base/tabs/TabStrip.java

Issue 29348083: Issue 3769 - UI colours gone bad (Closed)
Patch Set: Created July 21, 2016, 2:09 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mobile/android/base/resources/values/themes.xml ('k') | mobile/android/base/tabs/TabsPanel.java » ('j') | mobile/android/chrome/content/WebrtcUI.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mobile/android/base/tabs/TabStrip.java
===================================================================
--- a/mobile/android/base/tabs/TabStrip.java
+++ b/mobile/android/base/tabs/TabStrip.java
@@ -140,20 +140,22 @@ public class TabStrip extends ThemedLine
@Override
public void onLightweightThemeChanged() {
final Drawable drawable = getTheme().getDrawable(this);
if (drawable == null) {
return;
}
final StateListDrawable stateList = new StateListDrawable();
- stateList.addState(PRIVATE_STATE_SET, getColorDrawable(R.color.text_and_tabs_tray_grey));
+ // Using ABB color palette. See https://issues.adblockplus.org/ticket/3769
+ stateList.addState(PRIVATE_STATE_SET, getColorDrawable(R.color.abb_background_light_grey));
stateList.addState(EMPTY_STATE_SET, drawable);
setBackgroundDrawable(stateList);
}
@Override
public void onLightweightThemeReset() {
- final int defaultBackgroundColor = ColorUtils.getColor(getContext(), R.color.text_and_tabs_tray_grey);
+ // Using ABB color palette. See https://issues.adblockplus.org/ticket/3769
+ final int defaultBackgroundColor = ColorUtils.getColor(getContext(), R.color.abb_background_light_grey);
setBackgroundColor(defaultBackgroundColor);
}
}
« no previous file with comments | « mobile/android/base/resources/values/themes.xml ('k') | mobile/android/base/tabs/TabsPanel.java » ('j') | mobile/android/chrome/content/WebrtcUI.js » ('J')

Powered by Google App Engine
This is Rietveld