From 90a85bc89663b4877004053545add5da273f24f3 Mon Sep 17 00:00:00 2001 From: Damyan Ivanov Date: Sat, 26 Sep 2020 19:47:44 +0300 Subject: [PATCH] themes: secondary==primary, a bit closer primaryDark, no calculations all intermediate colours are calculated as a weighted average of the nearest theme color. this keeps the color calculations in one place -- tools/gen-styles.pl --- .../ui/activity/ProfileThemedActivity.java | 5 +- .../net/ktnx/mobileledger/utils/Colors.java | 184 ++++------- app/src/main/res/values-night/styles.xml | 296 +++++++++--------- app/src/main/res/values/styles.xml | 292 ++++++++--------- tools/gen-styles | 16 +- 5 files changed, 364 insertions(+), 429 deletions(-) diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/activity/ProfileThemedActivity.java b/app/src/main/java/net/ktnx/mobileledger/ui/activity/ProfileThemedActivity.java index d33608a6..81450c9a 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/activity/ProfileThemedActivity.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/activity/ProfileThemedActivity.java @@ -1,5 +1,5 @@ /* - * Copyright © 2019 Damyan Ivanov. + * Copyright © 2020 Damyan Ivanov. * This file is part of MoLe. * MoLe is free software: you can distribute it and/or modify it * under the term of the GNU General Public License as published by @@ -30,7 +30,8 @@ import net.ktnx.mobileledger.utils.Colors; public class ProfileThemedActivity extends CrashReportingActivity { protected MobileLedgerProfile mProfile; protected void setupProfileColors() { - Colors.setupTheme(this, mProfile); + final int themeHue = (mProfile == null) ? -1 : mProfile.getThemeHue(); + Colors.setupTheme(this, themeHue); } @Override protected void onStart() { diff --git a/app/src/main/java/net/ktnx/mobileledger/utils/Colors.java b/app/src/main/java/net/ktnx/mobileledger/utils/Colors.java index fdaacd5c..9d0c6aaa 100644 --- a/app/src/main/java/net/ktnx/mobileledger/utils/Colors.java +++ b/app/src/main/java/net/ktnx/mobileledger/utils/Colors.java @@ -23,58 +23,54 @@ import android.content.res.Resources; import android.util.TypedValue; import androidx.annotation.ColorInt; -import androidx.annotation.ColorLong; import androidx.annotation.NonNull; -import androidx.annotation.Nullable; import androidx.lifecycle.MutableLiveData; import net.ktnx.mobileledger.BuildConfig; import net.ktnx.mobileledger.R; -import net.ktnx.mobileledger.model.Data; import net.ktnx.mobileledger.model.MobileLedgerProfile; import net.ktnx.mobileledger.ui.HueRing; import java.util.ArrayList; import java.util.Collections; +import java.util.HashMap; import java.util.Locale; +import java.util.Objects; -import static java.lang.Math.abs; import static net.ktnx.mobileledger.utils.Logger.debug; public class Colors { public static final int DEFAULT_HUE_DEG = 261; - public static final int THEME_HUE_STEP_DEG = 5; - public static final int baseHueStep = 60; public static final MutableLiveData themeWatch = new MutableLiveData<>(0); - private static final float blueLightness = 0.665f; - private static final float yellowLightness = 0.350f; private static final int[][] EMPTY_STATES = new int[][]{new int[0]}; private static final int SWIPE_COLOR_COUNT = 6; + private static final int[] themeIDs = + {R.style.AppTheme_default, R.style.AppTheme_000, R.style.AppTheme_005, + R.style.AppTheme_010, R.style.AppTheme_015, R.style.AppTheme_020, R.style.AppTheme_025, + R.style.AppTheme_030, R.style.AppTheme_035, R.style.AppTheme_040, R.style.AppTheme_045, + R.style.AppTheme_050, R.style.AppTheme_055, R.style.AppTheme_060, R.style.AppTheme_065, + R.style.AppTheme_070, R.style.AppTheme_075, R.style.AppTheme_080, R.style.AppTheme_085, + R.style.AppTheme_090, R.style.AppTheme_095, R.style.AppTheme_100, R.style.AppTheme_105, + R.style.AppTheme_110, R.style.AppTheme_115, R.style.AppTheme_120, R.style.AppTheme_125, + R.style.AppTheme_130, R.style.AppTheme_135, R.style.AppTheme_140, R.style.AppTheme_145, + R.style.AppTheme_150, R.style.AppTheme_155, R.style.AppTheme_160, R.style.AppTheme_165, + R.style.AppTheme_170, R.style.AppTheme_175, R.style.AppTheme_180, R.style.AppTheme_185, + R.style.AppTheme_190, R.style.AppTheme_195, R.style.AppTheme_200, R.style.AppTheme_205, + R.style.AppTheme_210, R.style.AppTheme_215, R.style.AppTheme_220, R.style.AppTheme_225, + R.style.AppTheme_230, R.style.AppTheme_235, R.style.AppTheme_240, R.style.AppTheme_245, + R.style.AppTheme_250, R.style.AppTheme_255, R.style.AppTheme_260, R.style.AppTheme_265, + R.style.AppTheme_270, R.style.AppTheme_275, R.style.AppTheme_280, R.style.AppTheme_285, + R.style.AppTheme_290, R.style.AppTheme_295, R.style.AppTheme_300, R.style.AppTheme_305, + R.style.AppTheme_310, R.style.AppTheme_315, R.style.AppTheme_320, R.style.AppTheme_325, + R.style.AppTheme_330, R.style.AppTheme_335, R.style.AppTheme_340, R.style.AppTheme_345, + R.style.AppTheme_350, R.style.AppTheme_355, + }; + private static final HashMap themePrimaryColor = new HashMap<>(); public static @ColorInt int secondary; @ColorInt public static int tableRowDarkBG; public static int profileThemeId = -1; - private static final int[] themeIDs = - {R.style.AppTheme_000, R.style.AppTheme_005, R.style.AppTheme_010, R.style.AppTheme_015, - R.style.AppTheme_020, R.style.AppTheme_025, R.style.AppTheme_030, R.style.AppTheme_035, - R.style.AppTheme_040, R.style.AppTheme_045, R.style.AppTheme_050, R.style.AppTheme_055, - R.style.AppTheme_060, R.style.AppTheme_065, R.style.AppTheme_070, R.style.AppTheme_075, - R.style.AppTheme_080, R.style.AppTheme_085, R.style.AppTheme_090, R.style.AppTheme_095, - R.style.AppTheme_100, R.style.AppTheme_105, R.style.AppTheme_110, R.style.AppTheme_115, - R.style.AppTheme_120, R.style.AppTheme_125, R.style.AppTheme_130, R.style.AppTheme_135, - R.style.AppTheme_140, R.style.AppTheme_145, R.style.AppTheme_150, R.style.AppTheme_155, - R.style.AppTheme_160, R.style.AppTheme_165, R.style.AppTheme_170, R.style.AppTheme_175, - R.style.AppTheme_180, R.style.AppTheme_185, R.style.AppTheme_190, R.style.AppTheme_195, - R.style.AppTheme_200, R.style.AppTheme_205, R.style.AppTheme_210, R.style.AppTheme_215, - R.style.AppTheme_220, R.style.AppTheme_225, R.style.AppTheme_230, R.style.AppTheme_235, - R.style.AppTheme_240, R.style.AppTheme_245, R.style.AppTheme_250, R.style.AppTheme_255, - R.style.AppTheme_260, R.style.AppTheme_265, R.style.AppTheme_270, R.style.AppTheme_275, - R.style.AppTheme_280, R.style.AppTheme_285, R.style.AppTheme_290, R.style.AppTheme_295, - R.style.AppTheme_300, R.style.AppTheme_305, R.style.AppTheme_310, R.style.AppTheme_315, - R.style.AppTheme_320, R.style.AppTheme_325, R.style.AppTheme_330, R.style.AppTheme_335, - R.style.AppTheme_340, R.style.AppTheme_345, R.style.AppTheme_350, R.style.AppTheme_355, - }; public static void refreshColors(Resources.Theme theme) { TypedValue tv = new TypedValue(); theme.resolveAttribute(R.attr.table_row_dark_bg, tv, true); @@ -82,118 +78,54 @@ public class Colors { theme.resolveAttribute(R.attr.colorSecondary, tv, true); secondary = tv.data; - // trigger theme observers - themeWatch.postValue(themeWatch.getValue() + 1); - } - public static @ColorInt - int hslColor(float hueRatio, float saturation, float lightness) { - return 0xff000000 | hslTriplet(hueRatio, saturation, lightness); - } - public static @ColorInt - int hslTriplet(float hueRatio, float saturation, float lightness) { - @ColorLong long result; - float h = hueRatio * 6; - float c = (1 - abs(2f * lightness - 1)) * saturation; - float h_mod_2 = h % 2; - float x = c * (1 - Math.abs(h_mod_2 - 1)); - int r, g, b; - float m = lightness - c / 2f; - - if (h < 1 || h == 6) - return tupleToColor(c + m, x + m, 0 + m); - if (h < 2) - return tupleToColor(x + m, c + m, 0 + m); - if (h < 3) - return tupleToColor(0 + m, c + m, x + m); - if (h < 4) - return tupleToColor(0 + m, x + m, c + m); - if (h < 5) - return tupleToColor(x + m, 0 + m, c + m); - if (h < 6) - return tupleToColor(c + m, 0 + m, x + m); - - throw new IllegalArgumentException(String.format( - "Unexpected value for h (%1.3f) while converting hsl(%1.3f, %1.3f, %1.3f) to rgb", - h, hueRatio, saturation, lightness)); - } - public static @ColorInt - int tupleToColor(float r, float g, float b) { - int r_int = Math.round(255 * r); - int g_int = Math.round(255 * g); - int b_int = Math.round(255 * b); - return (r_int << 16) | (g_int << 8) | b_int; - } - public static float baseHueLightness(int baseHueDegrees) { - switch (baseHueDegrees % 360) { - case 0: - return 0.550f; // red - case 60: - return 0.250f; // yellow - case 120: - return 0.290f; // green - case 180: - return 0.300f; // cyan - case 240: - return 0.710f; // blue - case 300: - return 0.450f; // magenta - default: - throw new IllegalStateException( - String.format(Locale.US, "baseHueLightness called with invalid value %d", - baseHueDegrees)); + if (themePrimaryColor.size() == 0) { + for (int themeId : themeIDs) { + Resources.Theme tmpTheme = theme.getResources() + .newTheme(); + tmpTheme.applyStyle(themeId, true); + tmpTheme.resolveAttribute(R.attr.colorPrimary, tv, false); + themePrimaryColor.put(themeId, tv.data); + } } - } - public static float hueLightness(int hueDegrees) { - int mod = hueDegrees % baseHueStep; - int x0 = hueDegrees - mod; - int x1 = x0 + baseHueStep; - float y0 = baseHueLightness(x0); - float y1 = baseHueLightness(x1); - - return y0 + (hueDegrees - x0) * (y1 - y0) / (x1 - x0); + // trigger theme observers + themeWatch.postValue(themeWatch.getValue() + 1); } public static @ColorInt int getPrimaryColorForHue(int hueDegrees) { - int result = hslColor(hueDegrees / 360f, 0.845f, hueLightness(hueDegrees)); -// debug("colors", String.format(Locale.ENGLISH, "getPrimaryColorForHue(%d) = %x", -// hueDegrees, -// result)); - return result; - } - public static void setupTheme(Activity activity) { - MobileLedgerProfile profile = Data.getProfile(); - setupTheme(activity, profile); - } - public static void setupTheme(Activity activity, @Nullable MobileLedgerProfile profile) { - final int themeHue = (profile == null) ? -1 : profile.getThemeHue(); - setupTheme(activity, themeHue); + if (hueDegrees == DEFAULT_HUE_DEG) + return Objects.requireNonNull(themePrimaryColor.get(R.style.AppTheme_default)); + int mod = hueDegrees % HueRing.hueStepDegrees; + if (mod == 0) { + int themeId = getThemeIdForHue(hueDegrees); + Integer result = Objects.requireNonNull(themePrimaryColor.get(themeId)); + debug("colors", + String.format(Locale.US, "getPrimaryColorForHue(%d) = %x", hueDegrees, result)); + return result; + } + else { + int x0 = hueDegrees - mod; + int x1 = (x0 + HueRing.hueStepDegrees) % 360; + float y0 = Objects.requireNonNull(themePrimaryColor.get(getThemeIdForHue(x0))); + float y1 = Objects.requireNonNull(themePrimaryColor.get(getThemeIdForHue(x1))); + return Math.round(y0 + hueDegrees * (y1 - y0) / (x1 - x0)); + } } public static int getThemeIdForHue(int themeHue) { - int themeId = -1; + int themeIndex = -1; if (themeHue == 360) themeHue = 0; if ((themeHue >= 0) && (themeHue < 360) && (themeHue != DEFAULT_HUE_DEG)) { - int index; if ((themeHue % HueRing.hueStepDegrees) != 0) { Logger.warn("profiles", String.format(Locale.US, "Adjusting unexpected hue %d", themeHue)); - index = Math.round(1f * themeHue / HueRing.hueStepDegrees); + themeIndex = Math.round(1f * themeHue / HueRing.hueStepDegrees); } else - index = themeHue / HueRing.hueStepDegrees; - - themeId = themeIDs[index]; - } - - if (themeId < 0) { - themeId = R.style.AppTheme_default; - debug("profiles", - String.format(Locale.ENGLISH, "Theme hue %d not supported, using the default", - themeHue)); + themeIndex = themeHue / HueRing.hueStepDegrees; } - return themeId; + return themeIDs[themeIndex + 1]; // 0 is the default theme } public static void setupTheme(Activity activity, int themeHue) { int themeId = getThemeIdForHue(themeHue); @@ -292,10 +224,10 @@ public class Colors { chosenHue = (chosenIntervalStart + (largestInterval / 2)) % 360; } - final int mod = chosenHue % THEME_HUE_STEP_DEG; + final int mod = chosenHue % HueRing.hueStepDegrees; if (mod != 0) { - if (mod > THEME_HUE_STEP_DEG / 2) - chosenHue += (THEME_HUE_STEP_DEG - mod); // 13 += (5-3) = 15 + if (mod > HueRing.hueStepDegrees / 2) + chosenHue += (HueRing.hueStepDegrees - mod); // 13 += (5-3) = 15 else chosenHue -= mod; // 12 -= 2 = 10 } diff --git a/app/src/main/res/values-night/styles.xml b/app/src/main/res/values-night/styles.xml index de08d714..3d6fbf90 100644 --- a/app/src/main/res/values-night/styles.xml +++ b/app/src/main/res/values-night/styles.xml @@ -45,8 +45,8 @@ @@ -54,8 +54,8 @@ @@ -63,8 +63,8 @@ @@ -72,8 +72,8 @@ @@ -81,8 +81,8 @@ @@ -90,8 +90,8 @@ @@ -99,8 +99,8 @@ @@ -108,8 +108,8 @@ @@ -117,8 +117,8 @@ @@ -126,8 +126,8 @@ @@ -135,8 +135,8 @@ @@ -144,8 +144,8 @@ @@ -153,8 +153,8 @@ @@ -162,8 +162,8 @@ @@ -171,8 +171,8 @@ @@ -180,8 +180,8 @@ @@ -189,8 +189,8 @@ @@ -198,8 +198,8 @@ @@ -207,8 +207,8 @@ @@ -216,8 +216,8 @@ @@ -225,8 +225,8 @@ @@ -234,8 +234,8 @@ @@ -243,8 +243,8 @@ @@ -252,8 +252,8 @@ @@ -261,8 +261,8 @@ @@ -270,8 +270,8 @@ @@ -279,8 +279,8 @@ @@ -288,8 +288,8 @@ @@ -297,8 +297,8 @@ @@ -306,8 +306,8 @@ @@ -315,8 +315,8 @@ @@ -324,8 +324,8 @@ @@ -333,8 +333,8 @@ @@ -342,8 +342,8 @@ @@ -351,8 +351,8 @@ @@ -360,8 +360,8 @@ @@ -369,8 +369,8 @@ @@ -378,8 +378,8 @@ @@ -387,8 +387,8 @@ @@ -396,8 +396,8 @@ @@ -405,8 +405,8 @@ @@ -414,8 +414,8 @@ @@ -423,8 +423,8 @@ @@ -432,8 +432,8 @@ @@ -441,17 +441,17 @@ @@ -459,8 +459,8 @@ @@ -468,8 +468,8 @@ @@ -477,8 +477,8 @@ @@ -486,8 +486,8 @@ @@ -495,8 +495,8 @@ @@ -504,8 +504,8 @@ @@ -513,8 +513,8 @@ @@ -522,8 +522,8 @@ @@ -531,8 +531,8 @@ @@ -540,8 +540,8 @@ @@ -549,8 +549,8 @@ @@ -558,8 +558,8 @@ @@ -567,8 +567,8 @@ @@ -576,8 +576,8 @@ @@ -585,8 +585,8 @@ @@ -594,8 +594,8 @@ @@ -603,8 +603,8 @@ @@ -612,8 +612,8 @@ @@ -621,8 +621,8 @@ @@ -630,8 +630,8 @@ @@ -639,8 +639,8 @@ @@ -648,8 +648,8 @@ @@ -657,8 +657,8 @@ @@ -666,8 +666,8 @@ @@ -675,8 +675,8 @@ @@ -684,8 +684,8 @@ @@ -693,8 +693,8 @@ diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml index 9ac40fba..d7977654 100644 --- a/app/src/main/res/values/styles.xml +++ b/app/src/main/res/values/styles.xml @@ -49,8 +49,8 @@ @@ -58,8 +58,8 @@ @@ -67,8 +67,8 @@ @@ -76,8 +76,8 @@ @@ -85,8 +85,8 @@ @@ -94,8 +94,8 @@ @@ -103,8 +103,8 @@ @@ -112,8 +112,8 @@ @@ -121,8 +121,8 @@ @@ -130,8 +130,8 @@ @@ -139,8 +139,8 @@ @@ -148,8 +148,8 @@ @@ -157,8 +157,8 @@ @@ -166,8 +166,8 @@ @@ -175,8 +175,8 @@ @@ -184,8 +184,8 @@ @@ -193,8 +193,8 @@ @@ -202,8 +202,8 @@ @@ -211,8 +211,8 @@ @@ -220,8 +220,8 @@ @@ -229,8 +229,8 @@ @@ -238,8 +238,8 @@ @@ -247,8 +247,8 @@ @@ -256,8 +256,8 @@ @@ -265,8 +265,8 @@ @@ -274,8 +274,8 @@ @@ -283,8 +283,8 @@ @@ -292,8 +292,8 @@ @@ -301,8 +301,8 @@ @@ -310,8 +310,8 @@ @@ -319,8 +319,8 @@ @@ -328,8 +328,8 @@ @@ -337,8 +337,8 @@ @@ -346,8 +346,8 @@ @@ -355,8 +355,8 @@ @@ -364,8 +364,8 @@ @@ -373,8 +373,8 @@ @@ -382,8 +382,8 @@ @@ -391,8 +391,8 @@ @@ -400,8 +400,8 @@ @@ -409,8 +409,8 @@ @@ -418,8 +418,8 @@ @@ -427,8 +427,8 @@ @@ -436,8 +436,8 @@ @@ -445,8 +445,8 @@ @@ -454,8 +454,8 @@ @@ -463,8 +463,8 @@ @@ -472,8 +472,8 @@ @@ -481,8 +481,8 @@ @@ -490,8 +490,8 @@ @@ -499,8 +499,8 @@ @@ -508,8 +508,8 @@ @@ -517,8 +517,8 @@ @@ -526,8 +526,8 @@ @@ -535,8 +535,8 @@ @@ -544,8 +544,8 @@ @@ -553,8 +553,8 @@ @@ -562,8 +562,8 @@ @@ -571,8 +571,8 @@ @@ -580,8 +580,8 @@ @@ -589,8 +589,8 @@ @@ -598,8 +598,8 @@ @@ -607,8 +607,8 @@ @@ -616,8 +616,8 @@ @@ -625,8 +625,8 @@ @@ -634,8 +634,8 @@ @@ -643,8 +643,8 @@ @@ -652,8 +652,8 @@ @@ -661,8 +661,8 @@ @@ -670,8 +670,8 @@ @@ -679,8 +679,8 @@ @@ -688,8 +688,8 @@ @@ -697,8 +697,8 @@ diff --git a/tools/gen-styles b/tools/gen-styles index 25002e74..64cebf27 100644 --- a/tools/gen-styles +++ b/tools/gen-styles @@ -245,8 +245,8 @@ sub outputThemes { sub bestLightnessForHue { my ( $h, $s ) = @_; - my $targetContrast = 4.07; - my $white = Color::sRGB->WHITE; + my $targetContrast = $opt_night ? 5.16 : 4.07; + my $white = $opt_night ? Color::sRGB->BLACK : Color::sRGB->WHITE; my $bestLightness; my $bestContrast; for ( my $l = 0; $l < 1; $l += 0.002 ) { @@ -301,10 +301,12 @@ sub hslStyleForHue { # linear interpolation #my $l1 = $y0 + 1.0 * ( $hue - $x0 ) * ( $y1 - $y0 ) / ( $x1 - $x0 ); my $l1 = bestLightnessForHue( $hue / 360.0, $S ); + #$l1 += ( 1 - $l1 ) * 0.20 if $opt_night; - my $l2 = $l1 * 0.80; - my $l3 = $opt_night ? 0.150 : 0.950; - my $l4 = $opt_night ? 0.100 : 0.980; + #my $l2 = $opt_night ? ( $l1 + ( 1 - $l1 ) * 0.15 ) : $l1 * 0.85; + my $l2 = $l1 * 0.80; + my $l3 = $opt_night ? 0.150 : 0.950; + my $l4 = $opt_night ? 0.100 : 0.980; my $result = ""; my $indent = "$baseIndent "; @@ -328,9 +330,9 @@ sub hslStyleForHue { $result .= sprintf "$indent#00%s\n", 'colorPrimaryTransparent', hslHex( $hue, $S, $l1 ); $result .= sprintf "$indent#%s\n", - 'colorSecondary', hslHex( $hue, $S, $l2 ); + 'colorSecondary', hslHex( $hue, $S, $l1 ); $result .= sprintf "$indent#%s\n", - 'colorPrimaryDark', hslHex( $hue, $S, $l2 ); + 'colorPrimaryDark', hslHex( $hue, $S*0.8, $l2 ); $result .= sprintf "$indent#%s\n", 'table_row_dark_bg', hslHex( $hue, $S, $l3 ); $result .= sprintf "$indent#%s\n", -- 2.39.2