Skip to content

Commit 8ddf8c5

Browse files
authored
Merge pull request #1602 from michaelschattgen/feature/color-contrast-hidden-codes
Improve color contrast on hidden codes
2 parents ce29d12 + 3d50ab1 commit 8ddf8c5

File tree

3 files changed

+9
-1
lines changed

3 files changed

+9
-1
lines changed

app/src/main/java/com/beemdevelopment/aegis/ui/views/EntryHolder.java

+4-1
Original file line numberDiff line numberDiff line change
@@ -369,9 +369,11 @@ public void revealCode() {
369369
public void hideCode() {
370370
String code = getOtp();
371371
String hiddenText = code.replaceAll("\\S", Character.toString(HIDDEN_CHAR));
372+
stopExpirationAnimation();
373+
372374
updateTextViewWithDots(_profileCode, hiddenText, code);
373375
updateTextViewWithDots(_nextProfileCode, hiddenText, code);
374-
stopExpirationAnimation();
376+
375377
_hidden = true;
376378
}
377379

@@ -384,6 +386,7 @@ private void updateTextViewWithDots(TextView textView, String hiddenCode, String
384386
float dotsWidth = paint.measureText(hiddenCode);
385387
float scaleFactor = codeWidth / dotsWidth;
386388
scaleFactor = (float)(Math.round(scaleFactor * 10.0) / 10.0);
389+
textView.setTextColor(MaterialColors.getColor(textView, R.attr.colorCodeHidden));
387390

388391
// If scale is higher or equal to 0.8, do nothing and proceed with the normal text rendering
389392
if (scaleFactor >= 0.8) {

app/src/main/res/values/attrs.xml

+1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
<attr name="colorSuccess" />
99
<attr name="colorOnSurfaceDim" />
1010
<attr name="colorCode" />
11+
<attr name="colorCodeHidden" />
1112

1213
<declare-styleable name="SlideIndicator">
1314
<attr name="dot_radius" format="dimension" />

app/src/main/res/values/themes.xml

+4
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@
6262
<item name="colorSuccess">@color/aegis_theme_light_success</item>
6363
<item name="colorOnSurfaceDim">@color/aegis_theme_light_onSurfaceDim</item>
6464
<item name="colorCode">?attr/colorPrimary</item>
65+
<item name="colorCodeHidden">?attr/colorOutlineVariant</item>
6566
<!-- Intro colors -->
6667
<item name="dot_color">?attr/colorSurfaceVariant</item>
6768
<item name="dot_color_selected">?attr/colorOnSurfaceVariant</item>
@@ -132,6 +133,7 @@
132133
<item name="colorSuccess">@color/aegis_theme_dark_success</item>
133134
<item name="colorOnSurfaceDim">@color/aegis_theme_dark_onSurfaceDim</item>
134135
<item name="colorCode">?attr/colorPrimary</item>
136+
<item name="colorCodeHidden">?attr/colorOutlineVariant</item>
135137
<!-- Intro colors -->
136138
<item name="dot_color">?attr/colorSurfaceVariant</item>
137139
<item name="dot_color_selected">?attr/colorOnSurfaceVariant</item>
@@ -159,6 +161,7 @@
159161
<item name="colorSurfaceDim">#000000</item>
160162
<item name="colorSurfaceBright">#000000</item>
161163
<item name="colorCode">@android:color/white</item>
164+
<item name="colorCodeHidden">#2F2F2F</item>
162165
<item name="colorProgressbar">@android:color/white</item>
163166
</style>
164167

@@ -179,6 +182,7 @@
179182
<item name="colorSurfaceDim">#000000</item>
180183
<item name="colorSurfaceBright">#000000</item>
181184
<item name="colorCode">@android:color/white</item>
185+
<item name="colorCodeHidden">#2F2F2F</item>
182186
<item name="colorProgressbar">@android:color/white</item>
183187
</style>
184188

0 commit comments

Comments
 (0)