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

Unified Diff: mobile/android/base/toolbar/ShapedButtonFrameLayout.java

Issue 29348083: Issue 3769 - UI colours gone bad (Closed)
Patch Set: Adjusting comments which indicates what has being changed by this issue in upstream code Created Oct. 25, 2016, 12:16 p.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/toolbar/ShapedButton.java ('k') | mobile/android/base/toolbar/ToolbarEditLayout.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mobile/android/base/toolbar/ShapedButtonFrameLayout.java
===================================================================
--- a/mobile/android/base/toolbar/ShapedButtonFrameLayout.java
+++ b/mobile/android/base/toolbar/ShapedButtonFrameLayout.java
@@ -22,28 +22,30 @@ public class ShapedButtonFrameLayout ext
public ShapedButtonFrameLayout(Context context, AttributeSet attrs) {
super(context, attrs);
}
// The drawable is constructed as per @drawable/shaped_button.
@Override
public void onLightweightThemeChanged() {
- final int background = ColorUtils.getColor(getContext(), R.color.text_and_tabs_tray_grey);
+ // Using ABB color palette. See https://issues.adblockplus.org/ticket/3769
+ final int background = ColorUtils.getColor(getContext(), R.color.abb_background_light_grey);
final LightweightThemeDrawable lightWeight = getTheme().getColorDrawable(this, background);
if (lightWeight == null)
return;
lightWeight.setAlpha(34, 34);
final StateListDrawable stateList = new StateListDrawable();
stateList.addState(PRESSED_ENABLED_STATE_SET, getColorDrawable(R.color.highlight_shaped));
stateList.addState(FOCUSED_STATE_SET, getColorDrawable(R.color.highlight_shaped_focused));
- 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, lightWeight);
setBackgroundDrawable(stateList);
}
@Override
public void onLightweightThemeReset() {
setBackgroundResource(R.drawable.shaped_button);
« no previous file with comments | « mobile/android/base/toolbar/ShapedButton.java ('k') | mobile/android/base/toolbar/ToolbarEditLayout.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld