Feature: refactor activities

[komlan55]

• Included "MaterialDesign"
• Refactored NavigationDrawer
• Updated Fragments to Fragment support v4
• Updated Activity To AppCompatActivity
• Updated styles

[zaclimon]

C'est pas pire jusqu'à maintenant mais je trouve qu'il manque quelques affaires avant de le pousser notamment:

• Le titre après les Toolbars de MainActivity
• Une indication pour dire que la section "Profil" à changé de place (sur l'icone du user)
• Une nouvelle Toolbar quand on rentre dans un cours pour Moodle (Elle n'y est plus)
• Possiblement le thème pour le Drawer (Le thème en général semble light mais le thème du Drawer est dark)
• Changer le thème des dialogs. (Depuis que AppCompat semble être déjà utilisé, ça ne devrait pas être un probleme, au moins qu'une partie de Holo dégage XD) On dirait qu'on a des styles à partir de certains API mais qui ne semblent ne plus être nécessaire:

https://github.com/ApplETS/ETSMobile-Android2/blob/master/app/src/main/res/values-v11/styles.xml https://github.com/ApplETS/ETSMobile-Android2/blob/master/app/src/main/res/values-v14/styles.xml

Sinon j'ai quelques questions/suggestions (à ta discrétion bien entendu):

• Est-ce que l'utilisation des Chrome Custom Tabs va se faire dans ce PR?
• Possiblement augmenter le heap size lors de la compilation à 2G parce que MultiDex fait des tiennes... (Pourrait aussi fixer Travis)
• Changer le Target API/Build tools vers 25
• Remplacer le texte "Étudiant" dans le Drawer par le nom actuel de l'étudiant
• Possiblement mettre à jour quelques librairies que l'on utilise?
• Changer le Min API vers 19? (Je viendrai avec plus de détails éventuellement)

[ttauveron]

C'est un beau lifting pour l'app, ça la rajeunit beaucoup!

Par contre, ce n'est pas tout à fait prêt pour l'intégration selon moi.

• Sur mon téléphone, l'emploi du temps ne s'affiche pas, probablement à cause de l'erreur

java.net.ProtocolException: Too many follow-up requests: 21 at com.squareup.okhttp.Call.getResponse(Call.java:310) at com.squareup.okhttp.Call$ApplicationInterceptorChain.proceed(Call.java:230) at com.squareup.okhttp.Call.getResponseWithInterceptorChain(Call.java:201) at com.squareup.okhttp.Call.execute(Call.java:81) at ca.etsmtl.applets.etsmobile.http.AppletsApiCalendarRequest.loadDataFromNetwork(AppletsApiCalendarRequest.java:64) at ca.etsmtl.applets.etsmobile.http.AppletsApiCalendarRequest.loadDataFromNetwork(AppletsApiCalendarRequest.java:33)

Pour le drawer, c'est très bien trouvé de mettre des expandables, ça fait gagner beaucoup de place.

L'idée est bien d'avoir le profil dans l'entête du drawer et la photo de l'ÉTS en noir et blanc a un bon rendu

• Ça prendrait par contre un gradient pour faire ressortir le texte "Etudiant" et trouver une solution pour faire ressortir aussi l'icône du profil. Sur windows phone, Martin Gemme a lié gravatar par exemple (je lance une idée, ça pourrait être la photo de l'utilisateur sur moodle).

• Lorsque l'étudiant est loggué, ça serait bien d'afficher son prénom + nom à la place de "Étudiant".

• Ça prendrait également une traduction anglaise pour ce mot.

• C'est pas grand chose, mais ajouter aussi les événements dans le drawer.

• Comme Isaac l'a souligné, il y aurait probablement encore quelques modifications à ajouter au drawer, notamment pour les options de l'emploi du temps (export + switching calendrier/liste) et les titres des vues.

• Aussi mettre en blanc à la place de gris les items qui s'affichent pour les faire ressortir :

• Une petite confirmation pour le logout serait de mise également, dans le cas où l'utilisateur serait hors ligne et cliquerait accidentellement sur logout, il perdrait le cache de son emploi du temps en base de données.

Aussi, je ne suis pas capable de me reconnecter, il manque l'option !

• L'option pour se connecter est difficilement visible, ça serait bon de la faire ressortir et éventuellement, ajouter un lien dans l'entête du drawer quand l'étudiant n'est pas connecté et clique sur le profil.

[ttauveron]

Aussi le drop down est décalé ici :

[komlan55]

Damn, ca cest de commentaires détaillées lol! Merci guys 😄 !

[komlan55]

@zaclimon Oui ce serait bien que les Chrome custom tabs soit dans le PR. Peux tu t'en occuper?

[zaclimon]

C'est good je vais m'en occuper. Je vais juste finaliser ETSWebServices avant.