From b0ea7cecb503c332b03db160a6d149aecf5005e9 Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Mon, 15 Feb 2021 18:12:55 +0100
Subject: [PATCH] Move all colors to the same file and rename it

---
 CHANGES.md                                    |   1 +
 .../values/{colors_riotx.xml => colors.xml}   | 116 +++++++++++++++++-
 vector/src/main/res/values/colors_riot.xml    | 112 -----------------
 3 files changed, 112 insertions(+), 117 deletions(-)
 rename vector/src/main/res/values/{colors_riotx.xml => colors.xml} (72%)
 delete mode 100644 vector/src/main/res/values/colors_riot.xml

diff --git a/CHANGES.md b/CHANGES.md
index 514b0b9bdc..9096bee103 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -28,6 +28,7 @@ Test:
 Other changes:
  - New Dev Tools panel for developers
  - Fix typos in CHANGES.md (#2811)
+ - Colors rework: first step: merge file `colors_riot.xml` to file `colors_riotx.xml` and rename the file to `colors.xml`
 
 Changes in Element 1.0.17 (2021-02-09)
 ===================================================
diff --git a/vector/src/main/res/values/colors_riotx.xml b/vector/src/main/res/values/colors.xml
similarity index 72%
rename from vector/src/main/res/values/colors_riotx.xml
rename to vector/src/main/res/values/colors.xml
index 35d0c797b5..e4cfa18461 100644
--- a/vector/src/main/res/values/colors_riotx.xml
+++ b/vector/src/main/res/values/colors.xml
@@ -1,7 +1,115 @@
 <?xml version="1.0" encoding="utf-8"?>
 <resources>
 
-    <!-- Source: https://zpl.io/aBKw9Mk -->
+    <!-- Error colors -->
+    <color name="vector_success_color">#70BF56</color>
+    <color name="vector_warning_color">#ff4b55</color>
+    <color name="vector_error_color">#ff4b55</color>
+    <color name="vector_info_color">#2f9edb</color>
+
+    <!-- main app colors -->
+    <color name="vector_fuchsia_color">#ff4b55</color>
+    <color name="vector_silver_color">#FFC7C7C7</color>
+    <color name="vector_dark_grey_color">#FF999999</color>
+
+    <!-- theses colours are requested a background cannot be set by an ?attr on android < 5 -->
+    <!-- dedicated drawables are created for each theme -->
+    <!-- Default/Background-->
+    <color name="riot_primary_background_color_light">#FFFFFFFF</color>
+    <!-- Dark/Background-->
+    <color name="riot_primary_background_color_dark">#FF181B21</color>
+    <!-- Black/Background-->
+    <color name="riot_primary_background_color_black">#F000</color>
+
+    <!--Default/Android Status Bar-->
+    <color name="primary_color_dark_light">#FF1A2027</color>
+    <!--Default/Base-->
+    <color name="primary_color_light">#03b381</color>
+    <!--Default/Accent-->
+    <color name="accent_color_light">#03b381</color>
+
+    <!--Dark/Android Status Bar-->
+    <color name="primary_color_dark_dark">#FF0D0E10</color>
+    <!--Dark/Base-->
+    <color name="primary_color_dark">#FF15171B</color>
+    <!--Dark/Accent-->
+    <color name="accent_color_dark">#03b381</color>
+
+    <!--Black/Android Status Bar-->
+    <color name="primary_color_dark_black">#000</color>
+    <!--Black/Base-->
+    <color name="primary_color_black">#FF060708</color>
+
+    <!--Default/Line break mobile-->
+    <attr name="list_divider_color" format="color" />
+    <color name="list_divider_color_light">#EEEFEF</color>
+    <!--Dark/Line break mobile-->
+    <color name="list_divider_color_dark">#FF61708B</color>
+    <!--Black/Line break mobile-->
+    <color name="list_divider_color_black">#FF22262E</color>
+
+    <attr name="tab_bar_selected_background_color" format="color" />
+    <color name="tab_bar_selected_background_color_light">@color/riotx_android_secondary_light</color>
+    <color name="tab_bar_selected_background_color_dark">@color/riotx_android_secondary_dark</color>
+
+    <attr name="tab_bar_unselected_background_color" format="color" />
+    <color name="tab_bar_unselected_background_color_light">@color/riotx_background_light</color>
+    <color name="tab_bar_unselected_background_color_dark">@color/riotx_background_dark</color>
+
+    <!-- Hint Colors -->
+    <color name="primary_hint_text_color_light">#FFFFFF</color>
+    <color name="primary_hint_text_color_dark">#FFFFFF</color>
+
+    <color name="default_text_hint_color_light">#903C3C3C</color>
+    <color name="default_text_hint_color_dark">#CCDDDDDD</color>
+
+    <!-- Text Colors -->
+    <attr name="riot_primary_text_color" format="color" />
+    <attr name="riot_primary_text_color_disabled" format="color" />
+
+    <!--Default/Text Primary-->
+    <color name="riot_primary_text_color_light">#FF2E2F32</color>
+    <color name="riot_primary_text_color_disabled_light">#FF9E9E9E</color>
+    <!--Default/Text Secondary-->
+    <color name="riot_secondary_text_color_light">#FF9E9E9E</color>
+    <color name="riot_tertiary_text_color_light">@color/riot_primary_text_color_light</color>
+
+    <!--Dark /Text Primary-->
+    <color name="riot_primary_text_color_dark">#FFEDF3FF</color>
+    <color name="riot_primary_text_color_disabled_dark">#FFA1B2D1</color>
+    <!--Dark /Text Secondary-->
+    <color name="riot_secondary_text_color_dark">#FFA1B2D1</color>
+    <color name="riot_tertiary_text_color_dark">@color/riot_primary_text_color_dark</color>
+
+    <!-- Notification view colors -->
+    <color name="soft_resource_limit_exceeded">#2f9edb</color>
+    <color name="hard_resource_limit_exceeded">@color/vector_fuchsia_color</color>
+
+    <!-- Password Strength bar colors -->
+    <color name="password_strength_bar_weak">#FFF56679</color>
+    <color name="password_strength_bar_low">#FFFFC666</color>
+    <color name="password_strength_bar_ok">#FFF8E71C</color>
+    <color name="password_strength_bar_strong">#FF7AC9A1</color>
+    <color name="password_strength_bar_undefined">#FF9E9E9E</color>
+
+    <!-- Button color -->
+    <color name="button_enabled_text_color">#FFFFFFFF</color>
+    <color name="button_disabled_text_color">#FFFFFFFF</color>
+    <color name="button_destructive_enabled_text_color">#FF4B55</color>
+    <color name="button_destructive_disabled_text_color">#FF4B55</color>
+    <color name="button_bot_enabled_text_color">#FF368BD6</color>
+    <color name="button_bot_disabled_text_color">#61708B</color>
+
+
+    <!-- Link color -->
+    <color name="link_color_light">#368BD6</color>
+    <color name="link_color_dark">#368BD6</color>
+
+    <!-- Notification (do not depends on theme) -->
+    <color name="notification_accent_color">#368BD6</color>
+    <color name="key_share_req_accent_color">#ff812d</color>
+
+        <!-- Source: https://zpl.io/aBKw9Mk -->
 
     <!-- Accents -->
     <color name="riotx_accent">#FF0DBD8B</color>
@@ -38,7 +146,7 @@
     <color name="riotx_username_7">#5c56f5</color>
     <color name="riotx_username_8">#74d12c</color>
 
-    <!-- Other usefull color -->
+    <!-- Other useful color -->
     <color name="black">#FF000000</color>
     <color name="white">#FFFFFFFF</color>
     <color name="black_alpha">#55000000</color>
@@ -256,6 +364,4 @@
     <color name="riotx_keys_backup_banner_accent_color_light">#FFF8E3</color>
     <color name="riotx_keys_backup_banner_accent_color_dark">#22262E</color>
 
-
-
-</resources>
\ No newline at end of file
+</resources>
diff --git a/vector/src/main/res/values/colors_riot.xml b/vector/src/main/res/values/colors_riot.xml
deleted file mode 100644
index 2f9d521351..0000000000
--- a/vector/src/main/res/values/colors_riot.xml
+++ /dev/null
@@ -1,112 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<resources>
-
-    <!-- Error colors -->
-    <color name="vector_success_color">#70BF56</color>
-    <color name="vector_warning_color">#ff4b55</color>
-    <color name="vector_error_color">#ff4b55</color>
-    <color name="vector_info_color">#2f9edb</color>
-
-    <!-- main app colors -->
-    <color name="vector_fuchsia_color">#ff4b55</color>
-    <color name="vector_silver_color">#FFC7C7C7</color>
-    <color name="vector_dark_grey_color">#FF999999</color>
-
-    <!-- theses colours are requested a background cannot be set by an ?attr on android < 5 -->
-    <!-- dedicated drawables are created for each theme -->
-    <!-- Default/Background-->
-    <color name="riot_primary_background_color_light">#FFFFFFFF</color>
-    <!-- Dark/Background-->
-    <color name="riot_primary_background_color_dark">#FF181B21</color>
-    <!-- Black/Background-->
-    <color name="riot_primary_background_color_black">#F000</color>
-
-    <!--Default/Android Status Bar-->
-    <color name="primary_color_dark_light">#FF1A2027</color>
-    <!--Default/Base-->
-    <color name="primary_color_light">#03b381</color>
-    <!--Default/Accent-->
-    <color name="accent_color_light">#03b381</color>
-
-    <!--Dark/Android Status Bar-->
-    <color name="primary_color_dark_dark">#FF0D0E10</color>
-    <!--Dark/Base-->
-    <color name="primary_color_dark">#FF15171B</color>
-    <!--Dark/Accent-->
-    <color name="accent_color_dark">#03b381</color>
-
-    <!--Black/Android Status Bar-->
-    <color name="primary_color_dark_black">#000</color>
-    <!--Black/Base-->
-    <color name="primary_color_black">#FF060708</color>
-
-    <!--Default/Line break mobile-->
-    <attr name="list_divider_color" format="color" />
-    <color name="list_divider_color_light">#EEEFEF</color>
-    <!--Dark/Line break mobile-->
-    <color name="list_divider_color_dark">#FF61708B</color>
-    <!--Black/Line break mobile-->
-    <color name="list_divider_color_black">#FF22262E</color>
-
-    <attr name="tab_bar_selected_background_color" format="color" />
-    <color name="tab_bar_selected_background_color_light">@color/riotx_android_secondary_light</color>
-    <color name="tab_bar_selected_background_color_dark">@color/riotx_android_secondary_dark</color>
-
-    <attr name="tab_bar_unselected_background_color" format="color" />
-    <color name="tab_bar_unselected_background_color_light">@color/riotx_background_light</color>
-    <color name="tab_bar_unselected_background_color_dark">@color/riotx_background_dark</color>
-
-    <!-- Hint Colors -->
-    <color name="primary_hint_text_color_light">#FFFFFF</color>
-    <color name="primary_hint_text_color_dark">#FFFFFF</color>
-
-    <color name="default_text_hint_color_light">#903C3C3C</color>
-    <color name="default_text_hint_color_dark">#CCDDDDDD</color>
-
-    <!-- Text Colors -->
-    <attr name="riot_primary_text_color" format="color" />
-    <attr name="riot_primary_text_color_disabled" format="color" />
-
-    <!--Default/Text Primary-->
-    <color name="riot_primary_text_color_light">#FF2E2F32</color>
-    <color name="riot_primary_text_color_disabled_light">#FF9E9E9E</color>
-    <!--Default/Text Secondary-->
-    <color name="riot_secondary_text_color_light">#FF9E9E9E</color>
-    <color name="riot_tertiary_text_color_light">@color/riot_primary_text_color_light</color>
-
-    <!--Dark /Text Primary-->
-    <color name="riot_primary_text_color_dark">#FFEDF3FF</color>
-    <color name="riot_primary_text_color_disabled_dark">#FFA1B2D1</color>
-    <!--Dark /Text Secondary-->
-    <color name="riot_secondary_text_color_dark">#FFA1B2D1</color>
-    <color name="riot_tertiary_text_color_dark">@color/riot_primary_text_color_dark</color>
-
-    <!-- Notification view colors -->
-    <color name="soft_resource_limit_exceeded">#2f9edb</color>
-    <color name="hard_resource_limit_exceeded">@color/vector_fuchsia_color</color>
-
-    <!-- Password Strength bar colors -->
-    <color name="password_strength_bar_weak">#FFF56679</color>
-    <color name="password_strength_bar_low">#FFFFC666</color>
-    <color name="password_strength_bar_ok">#FFF8E71C</color>
-    <color name="password_strength_bar_strong">#FF7AC9A1</color>
-    <color name="password_strength_bar_undefined">#FF9E9E9E</color>
-
-    <!-- Button color -->
-    <color name="button_enabled_text_color">#FFFFFFFF</color>
-    <color name="button_disabled_text_color">#FFFFFFFF</color>
-    <color name="button_destructive_enabled_text_color">#FF4B55</color>
-    <color name="button_destructive_disabled_text_color">#FF4B55</color>
-    <color name="button_bot_enabled_text_color">#FF368BD6</color>
-    <color name="button_bot_disabled_text_color">#61708B</color>
-
-
-    <!-- Link color -->
-    <color name="link_color_light">#368BD6</color>
-    <color name="link_color_dark">#368BD6</color>
-
-    <!-- Notification (do not depends on theme) -->
-    <color name="notification_accent_color">#368BD6</color>
-    <color name="key_share_req_accent_color">#ff812d</color>
-
-</resources>