Dark mode - resolve #38 by FrancoisDuchene · Pull Request #52 · TGLuis/Youpidapp

FrancoisDuchene · 2021-09-15T14:22:31Z

Voilà une PR vu que c'est ce que tu veux Luis :P
J'ai fait un NightMode qui s'active via un paramètre dans le fragment associé. Ca change tous les layouts utilisés dans l'app et ça enregistre la préférence dans les sharedPreferences
J'en ai profité pour faire un lifting sur l'app en améliorant la lisibilité du thème clair

Jimvy

Observations :

Mon thème système est le mode sombre. Quand je démarre l'application, par défaut il prend le thème sombre, mais dans les réglages il indique que c'est le thème clair. Il faudrait avoir une option par défaut "thème natif".
Pour cette même raison, passer en thème clair requiert de d'abord passer sur le thème sombre, puis de passer sur le thème clair.
Enfin, restart l'application (la kill via la liste des apps récentes, puis la relancer via le launcher) ne conserve pas l'ancien réglage du thème, il se remet à la valeur par défaut (thème "clair" mais sombre si système sombre).

Jimvy · 2021-11-13T14:24:52Z

Autres bugs pour le moment présents :

en thème "natif" mais sombre, appuyer sur un bouton va jouer le son, mais à la fin il ne revient pas à la couleur du thème sombre, mais du thème clair. Passer en vrai thème sombre résout le problème, donc c'est sans doute une mauvaise sélection
apparemment le bouton aléatoire bug un peu : @FrancoisDuchene peux-tu me dire comment tu trigger ce bug ? Je n'arrive pas à le reproduire
quand on change de thème, toute l'activité est restart, ce qui remet l'app à la page d'accueil.

J'espère enfin y regarder aujourd'hui (au lieu de travailler sur Refinery City), notamment pour faire les choses suivantes :

essayer de réduire le nombre de fichiers : pour le moment les layouts défaut et "night" sont très similaires à part les couleurs mises dessus, je suis sûr qu'il y a moyen de déplacer ça dans les thèmes pour rendre le truc plus modulaire
mettre une option de thème système, qui est choisi par défaut, pour résoudre le problème du faux thème clair/sombre
regarder si c'est possible de faire le thème X-dark
regarder si on peut éviter un reboot de l'app : après tout, on arrive à gérer le changement d'orientation en gardant même la recherche en cours active, donc ça doit pas être impossible de au moins revenir sur l'écran des paramètres ou d'accélérer le temps de chargement.

Jimvy · 2021-11-14T19:41:25Z

Observations :

Quand l'app démarre, sur un téléphone avec un mode sombre, sans avoir de réglages : AppCompatDelegate indique que le thème est "MODE_NIGHT_UNSPECIFIED" (probablement car l'app ne le set jamais) ; getResources().getConfiguration().uiMode indique "UI_MODE_NIGHT_YES" (car le téléphone est en effet dans cette configuration) ; et le DataPErsistenceHelper dit que le thème est désactivé (car il n'a jamais été set).
Passage dans les settings : toujours la même chose
Changement vers thème sombre (car le thème clair n'est pas accessible, l'app croit qu'elle est en thème clair) : AppCompat dit "2" (MODE_NIGHT_YES), uiMode dit YES, DPH dit "true" : ils sont d'accord, les oiseaux chantent
Restart de l'app: OK, DPH retient la valeur, mais AppCompat reste en "UNSPECIFIED".
Changement vers thème clair : AppCompat dit "1" (MODE_NIGHT_NO), uiMode dit NO, DPH dit "false" : ils sont d'accord
Dès qu'on restart l'app, AppCompat va repasser en "UNSPECIFIED", et le thème repasser en thème sombre. C'est logique : dans le cas où on est en thème clair, DPH va renvoyer "false", donc on n'appelle pas AppCompat, donc il reste dans le thème par défaut, le thème sombre cassé.

Si le système est en thème clair (aïe les yeux) :

Au premier démarrage de l'app, AppCompat est en UNSPECIFIED. Aucun bug graphique n'a lieu
Si je passe en thème sombre, étrangement, maintenant, à tout démarrage de l'app il se remettra bien en thème sombre, avec AppCompat qui est en MODE_NIGHT_YES. Donc là, ça marche. D'ailleurs, l'app commence toujours par s'ouvrir comme si elle était en thème clair, avant de switch en thème sombre. C'est comme ça pour toutes les apps d'ailleurs, c'est juste que la Youpidapp met vraiment beaucoup de temps à se lancer (5x plus que Messenger, c'est dire...)

A mon avis l'application spécifie très mal les thèmes qu'elle supporte

TGLuis

Je vais tester ça sur mon gsm :) et j'attends tes réponses sur mes commentaires. A part ça 💯

TGLuis · 2021-11-27T14:05:15Z

+    fun activateNightMode() {
+        sharedPref.edit().putBoolean(NIGHTMODE_ACTIVATED, true).apply()
+    }
+
+    fun deactivateNightMode() {
+        sharedPref.edit().putBoolean(NIGHTMODE_ACTIVATED, false).apply()
+    }
+
+    fun isNightModeActivated() : Boolean {
+        return sharedPref.getBoolean(NIGHTMODE_ACTIVATED, false)
+    }


D'habitude quand il y a un night mode dans les app, il y a pas un mode "système default"?

TGLuis · 2021-11-27T14:07:23Z

+    <color name="colorPrimary">#EFA800</color>              <!-- text in black -->
+    <color name="colorPrimaryDark">#b77900</color>          <!-- text in black -->
+    <color name="colorPrimaryLight">#ffd949</color>         <!-- text in black -->
+    <color name="colorPrimaryDarkMode">#f7d97f</color>      <!-- text in black -->
+    <color name="colorSecondary">#8e24aa</color>            <!-- text in white -->
+    <color name="colorSecondaryDark">#5c007a</color>        <!-- text in white -->
+    <color name="colorSecondaryLight">#c158dc</color>       <!-- text in black -->
+    <color name="colorSecondaryDarkMode">#cd93d8</color>    <!-- text in black -->


Je comprends pas les commentaires pour "text in white/black" ?

TGLuis · 2021-11-27T14:46:32Z

Après test, de l'app, j'ai rien trouvé de plus que JM comme bug.
Par contre, je me disais que dans le thème sombre, la barre tout en haut avec la batterie, l'heure etc elle est de la même couleur que l'app alors que dans le thème clair il y a une distinction. Je sais plus c'est quel paramètre mais il faudrait changer ça ^^

FrancoisDuchene added 6 commits September 14, 2021 14:49

début night mode

df03561

night mode quasi complet (manque le drawer)

43f0770

coté UI - night mode complet

34d6d6e

coté prefs - night mode complet

47c57ec

petit cleanup

294bdf1

oups petit oubli

5ae8ba6

FrancoisDuchene added the Graphicz Graphical related issue label Sep 15, 2021

FrancoisDuchene added this to the v69.42 milestone Sep 15, 2021

FrancoisDuchene self-assigned this Sep 15, 2021

FrancoisDuchene linked an issue Sep 15, 2021 that may be closed by this pull request

Ajouter un thème sombre #38

Open

Jimvy reviewed Nov 2, 2021

View reviewed changes

TGLuis reviewed Nov 27, 2021

View reviewed changes

Navigation Menu

Dark mode - resolve #38 - #52

Dark mode - resolve #38#52
FrancoisDuchene wants to merge 6 commits into
masterfrom
dark_mode

FrancoisDuchene commented Sep 15, 2021

Uh oh!

Jimvy left a comment

Uh oh!

Jimvy commented Nov 13, 2021

Uh oh!

Jimvy commented Nov 14, 2021

Uh oh!

TGLuis left a comment

Uh oh!

TGLuis Nov 27, 2021

Uh oh!

TGLuis Nov 27, 2021

Uh oh!

TGLuis commented Nov 27, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

FrancoisDuchene commented Sep 15, 2021

Uh oh!

Jimvy left a comment

Choose a reason for hiding this comment

Uh oh!

Jimvy commented Nov 13, 2021

Uh oh!

Jimvy commented Nov 14, 2021

Uh oh!

TGLuis left a comment

Choose a reason for hiding this comment

Uh oh!

TGLuis Nov 27, 2021

Choose a reason for hiding this comment

Uh oh!

TGLuis Nov 27, 2021

Choose a reason for hiding this comment

Uh oh!

TGLuis commented Nov 27, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants