-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Jt bible #7
Jt bible #7
Conversation
Tu veux peut-être que je repasse sur le min sdk 14 ? |
Bjr. Comment fais je pour visualiser les changements..désolé si ma question paraît trivial. Je ne maîtrise pas assez git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est vraiment top. Je suis enthousiasmé par l'avancée du projet ! Je test sur mon téléphone dés que j'ai un peu plus de temps. En attendant, voici les premiers commentaires et questions.
Sur le fond, je préférerais ne faire un miroir que des textes en eux même, sans la police, les css, js et autres pages support de manière à mieux maîtriser l’intégration de notre coté et surtout limiter la taille finale.
Si tu as besoin d'aide la dessus, on peut en parler par mail. C'est quelque chose que j'ai déjà fait pour epitre.co ;)
app/build.gradle
Outdated
@@ -5,7 +5,7 @@ android { | |||
buildToolsVersion '27.0.0' | |||
defaultConfig { | |||
applicationId "co.epitre.aelf_lectures" | |||
minSdkVersion 14 | |||
minSdkVersion 19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour quelle raison as-tu dû augmenter la version ?
|
||
/** | ||
* Created by jean-tiare on 12/03/18. | ||
*/ | ||
|
||
public class SectionBibleFragment extends SectionFragmentBase { | ||
public SectionBibleFragment(){ | ||
// Required empty public constructor | ||
// http://kosalgeek.com/webview-fragment-android-studio/ §6 L18 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Merci pour la référence.
app/build.gradle
Outdated
@@ -5,7 +5,7 @@ android { | |||
buildToolsVersion '27.0.0' | |||
defaultConfig { | |||
applicationId "co.epitre.aelf_lectures" | |||
minSdkVersion 14 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pourquoi as-tu dû augmenter le minSdkVersion
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça permet d'activer le débogage distant de la webview dans Chrome. C'était utile pour corriger un problème d'affichage des menus. Je laisse @nath5394 donner plus de détails si tu veux :)
* Could handle back press. | ||
* @return true if back press was handled | ||
*/ | ||
public boolean onBackPressed() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Parfait, définition de l'interface de base 👍
|
||
|
||
// TODO : Fix shadow on "Autres Livres" dropdown menu not showing on real phone | ||
// TODO : Add support for pinch to zoom in Bible (or at least buttons zooming) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On peut intégrer une première bêta sans le zoom, et on regarde ensemble pour le mutualiser avec la webview des offices, si tu veux.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oui OK. Mon idée c'était de fournir rapidement une première intégration de la Bible qui marche. Et après on améliore.
// TODO : Fix shadow on "Autres Livres" dropdown menu not showing on real phone | ||
// TODO : Add support for pinch to zoom in Bible (or at least buttons zooming) | ||
// TODO : Test Bible on tablet ! | ||
// TODO : Link daily readings from mass and offices to Bible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ça peut faire l'objet d'une mise à jour indépendante.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pareil pour les 2 points suivants. Cette roadmap me plait !
|
||
// Enable remote debugging | ||
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT) { | ||
WebView.setWebContentsDebuggingEnabled(true); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tu pourras m'expliquer comment cela fonctionne ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En activant cela, tu peux accéder à un inspecteur web sur ton PC dans chrome (ou chromium) de la webview. Cela passe par adb. https://developers.google.com/web/tools/chrome-devtools/remote-debugging/webviews
} | ||
// Make the back button goes back in webview's history |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: s/goes/go/
|
||
List fragmentList = getSupportFragmentManager().getFragments(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je pense que l'on peut faire beaucoup plus simple ici. À priori, le seul fragment qui a besoin de recevoir l'evt, c'est celui qui est actuellement affiché. À ce moment, tu peux simplement faire quelquechose comme :
if (sectionFragment != null && sectionFragment.onBackPressed()) {
return;
}
super.onBackPressed();
Et dans un second temps, on pourra intégrer cela dans la "back stack" d'android pour revenir en arrière entre la Bible et les offices.
@@ -28,7 +28,8 @@ | |||
<activity | |||
android:name="co.epitre.aelf_lectures.LecturesActivity" | |||
android:resizeableActivity="true" | |||
android:label="@string/app_name" > | |||
android:label="@string/app_name" | |||
android:configChanges="orientation|screenSize"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Il faudra que je test. Pourquoi as-tu dû ajouter ce paramètre ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pour éviter que la webview soit rechargée quand on tourne le téléphone, ce qui nous faisait arriver sur la page d'accueil.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK. Du coup, si Android garbage collecte l'application, à l'ouverture suivante, on revient sur la page d'accueil aussi.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testé sur mon téléphone, cela donne un coté beaucoup plus fluide aux offices, c'est top !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je connais pas Android Garbage, tu peux m'indiquer une doc ou m'en dire plus ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Garbage --> garbage collector ;). Si Android a besoin de mémoire, il peut demander à tout instant à une activity qui n'est pas affichée de sauvegarder son état et se terminer. https://developer.android.com/guide/components/activities/activity-lifecycle.html
@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. |
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 ? |
En parcourant le diff, je note qu'il y a pas mal de fichiers et dossiers dans les ressources qui semblent inutiles. Par exemple:
Peut-être d'autre, à vérifier ! @nath5394 Tu peux les supprimer ? |
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). |
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:
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 ? |
|
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 🤣 😄 ; et puis j'ai hâte de pouvoir faire tester la bêta aux copains) |
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). |
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. |
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 ! |
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.) |
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. |
Ç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:
#7 (comment)
--
Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.
|
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 |
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:
#7 (comment)
--
Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.
|
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 ... |
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 ! |
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. |
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
…viewport units (vh/vw) with %.
Using sed -i -e '10d;17,59d;67,87d' ./*/*.html Signed-off-by: Nathanael <nathanael@hannebert.com>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
Signed-off-by: Jean-Tiare Le Bigot <jt@yadutaf.fr>
a945c14
to
9d63727
Compare
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 ! |
Salut, merci pour tes dernières modifications. Je suis d'accord : Go ! 👍 |
Hello,
Peux-tu me donner ton avis sur cette proposition ?
Nath