comments updated.

Author: syslogic

library/src/main/java/io/syslogic/colorpicker/AlphaPatternDrawable.java

@@ -31,8 +31,8 @@ public class AlphaPatternDrawable extends Drawable {
 
     AlphaPatternDrawable(int rectangleSize) {
         this.mRectangleSize = rectangleSize;
-        this.mPaintWhite.setColor(0xffffffff);
-        this.mPaintGray.setColor(0xffcbcbcb);
+        this.mPaintWhite.setColor(0xFFFFFFFF);
+        this.mPaintGray.setColor(0xFFCBCBCB);
     }
 
     @Override

library/src/main/java/io/syslogic/colorpicker/ColorPickerPanelView.java

@@ -23,36 +23,28 @@ public class ColorPickerPanelView extends View {
     private int mColor = 0xFF000000;
 
     /** the width in pixels of the border surrounding the color panel. */
-    private float BORDER_WIDTH_PX;
+    private final float BORDER_WIDTH_PX;
 
-    private float mDensity = 1f;
-
-    private Paint mBorderPaint;
-    private Paint mColorPaint;
+    private final Paint mBorderPaint;
+    private final Paint mColorPaint;
 
     private RectF mDrawingRect;
     private RectF mColorRect;
 
     private AlphaPatternDrawable mAlphaPattern;
 
+    private final float mDensity;
+
     public ColorPickerPanelView(@NonNull Context context, @NonNull AttributeSet attrs) {
         this(context, attrs, 0);
     }
 
     public ColorPickerPanelView(@NonNull Context context, @NonNull AttributeSet attrs, int defStyle) {
         super(context, attrs, defStyle);
-        init(context);
-    }
-
-    private void init(@NonNull Context context) {
-        this.mDensity = getDisplayDensity(context);
         BORDER_WIDTH_PX = getDimension(context, R.dimen.border_width_px);
-        this.initPaintTools();
-    }
-
-    private void initPaintTools() {
-        mBorderPaint = new Paint();
-        mColorPaint = new Paint();
+        this.mDensity = getDisplayDensity(context);
+        this.mBorderPaint = new Paint();
+        this.mColorPaint = new Paint();
     }
 
     private static float getDimension(@NonNull Context context, @DimenRes int resId) {
@@ -111,7 +103,6 @@ private void setUpColorRect() {
 
     /**
      * Set the color that should be shown by this view.
-     *
      * @param color the color value to set.
      */
     public void setColor(int color) {
@@ -121,7 +112,6 @@ public void setColor(int color) {
 
     /**
      * Get the color currently shown by this view.
-     *
      * @return the current color value.
      */
     public int getColor() {
@@ -130,7 +120,6 @@ public int getColor() {
 
     /**
      * Set the color of the border surrounding the panel.
-     *
      * @param color the color value to set.
      */
     @SuppressWarnings("unused")
@@ -141,7 +130,6 @@ public void setBorderColor(int color) {
 
     /**
      * Get the color of the border surrounding the panel.
-     *
      * @return the current border color value.
      */
     @SuppressWarnings("unused")

library/src/main/java/io/syslogic/colorpicker/ColorPickerPreference.java

@@ -170,6 +170,8 @@ public void setHexValueEnabled(boolean value) {
     }
 
     /**
+     *  String.format("#%1$02X", color)
+     *
      * @param color the color value to convert.
      */
     @NonNull

library/src/main/java/io/syslogic/colorpicker/compose/ColorPickerComponent.kt

@@ -470,10 +470,10 @@ fun getHSV(value: Color): FloatArray {
 
 /**
  * Convert H/S/V to Int.
- * @param alpha the color to convert.
- * @param hue the color to convert.
- * @param sat the color to convert.
- * @param value the color to convert.
+ * @param alpha the alpha of the color.
+ * @param hue the hue of the color.
+ * @param sat the saturation of the color.
+ * @param value the value of the color.
  * @return integer color value.
  */
 fun toIntColor(alpha: Float, hue: Float, sat: Float, value: Float): Int {