Jt bible - Githubissues

nathanael-h commented 6 years ago

Hello, Peux-tu me donner ton avis sur cette proposition ? Nath

nathanael-h commented 6 years ago

Tu veux peut-être que je repasse sur le min sdk 14 ?

randolphgamo commented 6 years ago

Bjr. Comment fais je pour visualiser les changements..désolé si ma question paraît trivial. Je ne maîtrise pas assez git

yadutaf commented 6 years ago

@randolphgamo les changements sont visibles dans https://github.com/HackMyChurch/aelf-dailyreadings/pull/7/files et sinon, l'ensemble du code est ici: https://github.com/nath5394/aelf-dailyreadings/tree/jt-bible.

AMDG2 commented 6 years ago

Concernant l'intégration des fichiers sources pour la bible que propose @yadutaf pour éviter de tout copier en limitant le travail d'intégration pour le moment ?

AMDG2 commented 6 years ago

En parcourant le diff, je note qu'il y a pas mal de fichiers et dossiers dans les ressources qui semblent inutiles. Par exemple:

app/src/main/assets/www/abonnement.html
app/src/main/assets/www/contact.html
app/src/main/assets/www/favicon.ico
app/src/main/assets/www/index_old.html
app/src/main/assets/www/mailing
app/src/main/assets/www/original_files
app/src/main/assets/www/page
app/src/main/assets/www/cdn-cgi

Peut-être d'autre, à vérifier ! @nath5394 Tu peux les supprimer ?

nathanael-h commented 6 years ago

Oui faudrait que je supprime ces fichiers inutiles.

Par ailleurs j'ai une version de la Bible en fichiers texte bruts : un fichier par livre, une ligne par verset. Je pense utiliser ce type de source de données à l'avenir car plus léger. Et passer sur du natif et non du web. Mais ça me demande plus de travail et donc je préfére sortir une version rapidement en web (petite satisfaction qui donne envie de continuer).

yadutaf commented 6 years ago

J'ai pris le temps de tester sur mon téléphone, le rendu visuel est agréable. Je pense que l'on a une bonne première version. J'ai hâte de proposer cela !

De mon coté, j'ai récemment ajouté un thème sombre alias "mode nuit" à l'application, en faisant attention à ce que cela ne vous complique pas trop la tâche.

Ce que je vous propose:

Je sors une "bêta 3" avec le nouveau thème, sans la Bible
Puis on intègre ce code dans l'application, avec un minSDK à 14, si ça ne pose pas de problème majeur en gardant le reste tel qu'il est. J'ai testé le "rebase" sur origin/master en local, cela passe sans difficultés (Alléluia)
Puis (vous allez me bouffer :fearful:) intégration du mode nuit
Puis "bêta 4"
Test, test, test

Avec cela, on peut sortir une première version 2.0 !

Ensuite, si cela vous va, on pourra travailler sur le mode de stockage de la Bible pour être à la fois plus petit, plus souple et mieux intégré.

Ça vous va ?

nathanael-h commented 6 years ago

Les fichiers inutiles c'est bon.
Le SDK min est repassé sur 14 Je commit ça ce soir. Le reste je sais pas quand je le ferai.

nathanael-h commented 6 years ago

Ah moi j'aurais priorisé autrement, d'abord la Bible et après le mode nuit ;) (Dans ma tête je me dis que c'est plus cool et utile d'abord dans sa poche une Bible qu'un mode nuit :rofl: :smile: ; et puis j'ai hâte de pouvoir faire tester la bêta aux copains) Bon c'est toi qui voit bien sûr :wink: Si tu restes sur ta proposition, cela m'ira très bien aussi, tant qu'on avance y'a pas de problème.

AMDG2 commented 6 years ago

Je veux bien m'occuper de l'intégration du mode nuit, puis pars la suite du stockage amélioré de la bible (sauf si @nath5394 veut s'en occuper).

nathanael-h commented 6 years ago

Salut, je sera bien intéressé de voir comment tu ajoutes le mode nuit à la Bible, parce que là comme ça je ne saurais pas trop dans quelle direction partir. Quant au changement de types de fichiers sources je veux bien le faire avec un peu d'aide. On en reparle, OK ? @yadutaf peux-tu mettre à jour la branche jt-bible depuis master sur ton dépôt ?

nathanael-h commented 6 years ago

Salut, j'ai ajouté 2 commits pour le support du thème sombre sur la partie Bible. Il semble que ça fonctionne sur le principe, par contre les CSS ne sont pas encore bonnes, je dois les corriger. J'accepte votre aide et vos conseils. Poke @yadutaf @AMDG2

yadutaf commented 6 years ago

C'est génial ! J'essaye de regarder dans la semaine. Je n'ai pas passé beaucoup de temps sur l'application ces derniers temps, mais il faut absolument publier cette immense amélioration !

nathanael-h commented 6 years ago

Merci pour ton enthousiasme :) j'essaie d'avancer sur les CSS qui à ce jour ne sont pas bonnes. (En fait c'est surtout la css dark_thème qui doit être corrigée et allégée de tout ce qui fait doublon.)

nathanael-h commented 6 years ago

Que pensez-vous de cette façon de faire : https://alligator.io/css/theming-custom-properties/ ? C'est différent de ce que j'ai commencé à faire.

AMDG2 commented 6 years ago

Ça m'a l'air parfait 👌

Le 19 juin 2018 22:51:59 GMT+02:00, nath5394 notifications@github.com a écrit :

Que pensez-vous de cette façon de faire : https://alligator.io/css/theming-custom-properties/ ? C'est différent de ce que j'ai commencé à faire.

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/HackMyChurch/aelf-dailyreadings/pull/7#issuecomment-398541532

-- Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.

yadutaf commented 6 years ago

Hmm, j'ai peur que cela ne fonctionne pas avec les téléphones les plus anciens, avant Android 4.4 (avant que la webview ne soit mise à jour automatiquement dans Android) cf https://caniuse.com/#feat=css-variables

nathanael-h commented 6 years ago

OK, je reste sur ce que j'ai déjà fait. J'essaie de corriger la css dark. Merci 😊

Le 19 juin 2018 23:07:35 GMT+02:00, Jean-Tiare Le Bigot notifications@github.com a écrit :

Hmm, j'ai peur que cela ne fonctionne pas avec les téléphones les plus anciens, avant Android 4.4 (avant que la webview ne soit mise à jour automatiquement dans Android) cf https://caniuse.com/#feat=css-variables

-- You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/HackMyChurch/aelf-dailyreadings/pull/7#issuecomment-398546247

-- Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.

nathanael-h commented 6 years ago

I rewrote the dark theme css. It's almost Ok. Can you send me the bugs you have on your phones ? It's not rendering identically on phone and PC ...

AMDG2 commented 6 years ago

C'est vachement bien \o/

Je vois trois problèmes, le premier c'est sur la page de sélection du livre, j'ai du beige qui est encore là: screenshot_20180621-075639

Le second problème c'est qu'on ne distingue pas bien la limite du sélecteur de livre avec le reste de la page: screenshot_20180621-075648

Le troisième problème est plus une question de gout, je trouve le rouge qui indique la sélection en cours et les numéros de verset (il me semble que ce sont les mêmes) trop rose.

Bravo pour le super boulot en tout cas :raised_hands:

nathanael-h commented 6 years ago

Voilà il me semble que le thème sombre est complet pour la partie Bible, qu'il n'y a plus de bug. Dites-moi si vous préférez d'autres choix de couleurs pour certains éléments de l'interface. Si ça te convient en l'état @yadutaf eh bien merge 😀😀et sortons une beta 🙏😉

J'ai hâte !

AMDG2 commented 6 years ago

Le rouge à changé, je le trouve moins rose, mais je pense qu'il serait mieux de la même couleur que le bandeau "Bible". Après pour le reste tout est bon chez moi, @yadutaf c'est toi le chef, à toi de voir pour les couleurs.

yadutaf commented 5 years ago

Hello !

En ce qui me concerne, on est bon pour passer en "bêta". Il restera des ajustements de thème / ménage que l'on fera dans un second temps.

Dès que j'ai votre Go, on y va.

Merci !

nathanael-h commented 5 years ago

Salut, merci pour tes dernières modifications. Je suis d'accord : Go ! :+1:

HackMyChurch / aelf-dailyreadings

Jt bible #7