Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Unified Diff: ui/android/java/src/org/chromium/ui/ColorPickerDialog.java

Issue 23026006: Add support for color input datalist on Android (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/android/java/src/org/chromium/ui/ColorPickerDialog.java
diff --git a/ui/android/java/src/org/chromium/ui/ColorPickerDialog.java b/ui/android/java/src/org/chromium/ui/ColorPickerDialog.java
index 58f1e0a30237231d52e23b612b4c5c17a3ee7aa7..ca4d245f5b1caf2cb1aa6dd05c1ed070fd722fc1 100644
--- a/ui/android/java/src/org/chromium/ui/ColorPickerDialog.java
+++ b/ui/android/java/src/org/chromium/ui/ColorPickerDialog.java
@@ -38,8 +38,12 @@ public class ColorPickerDialog extends AlertDialog implements OnColorChangedList
* @param theme The theme to display the dialog in.
* @param listener The object to notify when the color is set.
* @param color The initial color to set.
+ * @param suggestions The list of suggestions.
*/
- public ColorPickerDialog(Context context, OnColorChangedListener listener, int color) {
+ public ColorPickerDialog(Context context,
+ OnColorChangedListener listener,
+ int color,
+ ColorSuggestion[] suggestions) {
super(context, 0);
mListener = listener;
@@ -110,7 +114,7 @@ public class ColorPickerDialog extends AlertDialog implements OnColorChangedList
// Initialize simple color view (default view).
mSimpleColorPicker = (ColorPickerSimple) content.findViewById(R.id.color_picker_simple);
- mSimpleColorPicker.init(this);
+ mSimpleColorPicker.init(suggestions, this);
updateCurrentColor(mInitialColor);
}
@@ -136,8 +140,8 @@ public class ColorPickerDialog extends AlertDialog implements OnColorChangedList
View buttonBorder = findViewById(R.id.more_colors_button_border);
buttonBorder.setVisibility(View.GONE);
- View simpleViewBorder = findViewById(R.id.color_picker_simple_border);
- simpleViewBorder.setVisibility(View.GONE);
+ View simpleView = findViewById(R.id.color_picker_simple);
+ simpleView.setVisibility(View.GONE);
mAdvancedColorPicker.setVisibility(View.VISIBLE);
mAdvancedColorPicker.setListener(this);

Powered by Google App Engine
This is Rietveld 408576698