Proposer un template basique pour la création de thème

[marienfressinaud]

En amont de #120, il serait bien de proposer un template de design très simple qu'il suffira d'étendre pour créer un nouveau thème

Penser aussi à documenter les classes utiles sur le wiki

[marienfressinaud]

J'ai fini le template vraiment basique. Je n'avais pas prévu qu'il soit utilisable mais il l'est :) J'aimerais encore le réduire pour arriver à ~500 lignes (je ne sais pas si ce sera facile)

Lorsqu'il sera finalisé, ce serait bien de baser tous les thèmes officiels dessus. C'est-à-dire :

• au lieu d'avoir deux fichiers global.css et freshrss.css, on aura template.css et specific.css (ou quelque chose comme ça)
• il ne faudra pas toucher au template sauf si la modification doit être reporté sur tous les autres thèmes.

Ça devrait faciliter les mises à jour des designs. Bien sûr un thème n'aura pas la nécessité de se baser sur le template, mais ce sera un gage de "qualité" :)

Pour procéder j'ai :

• Copié/collé le thème Origine dans un fichier
• J'ai enlevé toutes les propriétés CSS qui ne correspondaient pas à : position, top/bottom/left/right, display, *height, *width, overflow, z-index et quelques autres.
• J'ai gardé en triant sur le volet pour ce qui est de : border*, background, margin/padding et quelques autres.

Maintenant il reste peut-être à refactoriser le code et virer les propriétés facultatives (je suis sûr qu'il y en a). Puisque le template est actuellement utilisable tel quel, ce serait bien qu'il le reste ;)

[marienfressinaud]

Le dernier commit restructure le fichier template, ajoute des commentaires et supprime encore quelques règles (~70 lignes supprimées). Ça va être dur d'en supprimer encore plus sans perdre niveau visuel.

Il reste à transformer les unités px en em ou rem (#324 (comment)) au niveau du texte.

[marienfressinaud]

Encore quelques lignes supprimées :) De plus j'ai changé la majorité des unités en "em" comme proposé dans #324. J'ai quand même gardé quelques px car ils me semblaient pertinents.

Avec ce dernier patch j'ai envie de dire que le template est prêt :)

[aledeg]

Quand penses tu le merger dans dev?

[marienfressinaud]

En fait je ne pense pas merger le template : il s'agit juste d'une base pour aider à la création de thèmes. Ce qui sera mergé ce sont les thèmes basés sur ce template

[aledeg]

si je comprends bien, tu vas reprendre l'ensemble des thèmes existant pour les migrer sur le nouveau template?

[marienfressinaud]

C'est exactement ça... mais j'avoue que ces derniers jours je n'ai pas la foi de m'y mettre :(

[aledeg]

ok, ça arrive :)

[marienfressinaud]

Le commit 1fdd76b pose les bases de la refonte du thème Origine. Il n'est pas terminé et il y a des soucis au niveau de la taille de police des boutons-dropdown, mais depuis le temps que ça traine, je push !

[aledeg]

👍
As tu converti par la même occasion les thèmes existant?

[marienfressinaud]

Non justement, c'est ce que j'ai commencé avec le dernier commit, mais il y a pas mal de boulot. Les autres thèmes étant basés sur Origine seront plus faciles à migrer à priori

[aledeg]

Ok. Bon courage alors.

[marienfressinaud]

En 10 minutes je viens de corriger ce sur quoi je bloquais la dernière fois :D J'ai bon espoir maintenant ;)

[marienfressinaud]

Et voilà, le thème Origine2 devrait être totalement fonctionnel ! :) Attention, j'ai fait de toutes petites modifs au niveau des notifications et de la boîte qui affiche les infos d'actualisation des flux. Je checkerai une dernière fois demain et remplacerai le thème Origine de la branche dev par le thème Origine2 de la branche 320-template si tout est ok.

[marienfressinaud]

Après une utilisation rapide, quelques points à modifier :

• La barre de déconnexion en vue mobile n'est pas top
• Le bouton de fermeture des notifications doit s'étendre sur toute la longueur de la notif en vue mobile
• Les titres sont mal tronqués (pas de points de suspension) -> se pose la question du système de "révélation" des titres tronqués à leur survol : les titres peuvent se mettre sur plusieurs lignes et survoler les titres en-dessous mais personnellement je ne trouve pas ça super pratique
• Il faudra ajouter une option pour choisir la largeur sur laquelle doit s'étendre le contenu des articles (4 options : "thin", "medium", "large" et "no-limit"). C'est déjà pris en charge dans le template de base
• Revoir les pages de stats, logs et à propos
• Les dropdowns partager et tags sont invisibles (en-dessous des titres qui suivent, se repose la question du système de survol des titres)

[marienfressinaud]

Deux nouveaux trucs à corriger :

• L'affichage des citations et du code source
• La popup affichant l'actualisation des flux doit suivre le défilement de l'écran

[marienfressinaud]

Le dernier commit corrige tous les problèmes que j'ai pu relever ! J'ai fait quelques améliorations encore par rapport au thème de base, ça commence à devenir bien sympa :)

[marienfressinaud]

Encore un petit soucis : la notification lorsqu'il n'y a aucun flux à afficher ne s'affiche pas (juste une histoire d'id / class html)

[marienfressinaud]

J'ai corrigé deux trois autres trucs ce soir et j'ai changé le nom Origine2 en Origine (version 0.2). Ça reste dans la branche 320-template pour le moment mais ça ne va pas tarder à pouvoir être mergé... Encore que j'hésite à attendre la migration des deux autres thèmes sur le template.

Je vais faire quelques petites modifs sur la branche dev au niveau des boutons pour améliorer des choses avant.

[marienfressinaud]

Et voilà ! \o/ C'est tout dans la branche dev :) Y a un petit paquet d'améliorations au niveau de l'interface, j'espère que vous aimerez.

Il ne reste plus qu'à migrer les deux autres thèmes sur le template de base mais ils semblent encore fonctionnels.

[marienfressinaud]

Encore des petits détails :

• Enlever le margin left et right des blockquotes en vue mobile
• Grossir la bordure gauche des articles en vue mobile
• Virer les pointillés pour certains liens ?
• "Sticker" l'input et le bouton pour révéler le mot de passe API

[Jucgshu]

Je suis en train de tester les derniers changements. Je suis très fan de la nouvelle fonction pour jouer avec la largeur du contenu !

👏

[aledeg]

👍 Je viens de regarder ça et c'est vraiment bien.
Par contre, je viens de remarquer que l'affichage des statistiques est cassé avec le nouveau thème.

[aledeg]

On dirait aussi que le thème de l'installeur est cassé. À confirmer. À cette heure, je ne suis plus très sur.

[marienfressinaud]

Les stats ne sont pas cassés, j'ai juste enlevé les ombres : je trouvais que ça n'allait pas avec le reste du thème. J'ai aussi essayé de revoir les tableaux mais peut-être que le contraste des couleurs n'est pas suffisant.

Pour l'installateur je n'ai pas regardé (shame on me!). Je corrigerai ça ce soir s'il y a lieu

[aledeg]

Pour les stats, il n'y a pas qu'un problème d'ombres. Il y a aussi le contenu de certaines colonnes qui est centré au lieu d'être aligné à droite dans le top ten.

[marienfressinaud]

Ah oui, je voulais rajouter une classe "numeric" aux cases avec des chiffres. Celles-ci seront alignées à droite (ou centrées ?) tandis que les autres seront alignés à gauche.

[aledeg]

À droite, ce serait mieux.

[marienfressinaud]

J'ai corrigé pour l'installateur. Pour ce qui est des statistiques on corrigera ça dans ton patch :)

[aledeg]

ok

[marienfressinaud]

Je vais attaquer la migration des deux autres thèmes mais comme le template est relativement stable désormais, je ferme ce ticket ;)

[marienfressinaud]

Et voilà ! Tous les thèmes sont basés sur le template de base ! :) Flat et Dark ont peut-être subis quelques régressions, mais le gros du travail a été fait.