Skip to content
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

Questions : à propos des thèmes (UI) #533

Closed
GLLM opened this issue Jul 9, 2014 · 15 comments
Closed

Questions : à propos des thèmes (UI) #533

GLLM opened this issue Jul 9, 2014 · 15 comments
Labels
Milestone

Comments

@GLLM
Copy link

GLLM commented Jul 9, 2014

Salut,

j'utilise freshRSS depuis 1 semaine et j'en suis ravi.
Super boulot, bravo !! c'est nettement mieux que tous les autres trucs auto-hébergés que j'ai utilisé/testé...

J'ai une question relative aux thèmes car que j'ai commencé à créer un thème et je me pose quelques questions :

  • y a t il un thème à utiliser comme "base" ?
  • les modif que j'apporte (surtout des modif de couleurs et de taille) sont-elles une peu durables, ou la prochaine modif structurelles sur les thèmes vont-elles tout casser ?
  • certains item .content{} sont les mêmes quelque soit le device utilisé (smartphone, PC). Une ségregation avec @media pourrait-elle être utile ?

Voilà un petit screenshot de ce que j'ai fait :
theme-freshrss

Si ca intéresse certains, faîtes moi signe.

A+ & merci,
GLLM

@Alkarex Alkarex added UI labels Jul 9, 2014
@Alkarex Alkarex added this to the 0.8.0 milestone Jul 9, 2014
@aledeg
Copy link
Member

aledeg commented Jul 9, 2014

C'est ce que @marienfressinaud a développé récemment.
Voir #320

@Alkarex
Copy link
Member

Alkarex commented Jul 9, 2014

@GLLM Semble bien parti 👍 N'hésitez-pas à poster des captures d'écran de la progression.

@aledeg ou quelqu'un d'autre : Ça pourrait être bien avec une petite phrase au sujet du développement des thèmes dans la doc, si ça n'y est pas déjà.

@aledeg
Copy link
Member

aledeg commented Jul 9, 2014

Je pense que @marienfressinaud est le mieux placé pour faire une bafouille dans la doc

@marienfressinaud
Copy link
Member

Je te réponds complètement dès que je suis rentré chez moi :) en bref, le mieux est de se baser sur le fichier template.css : c'est le même quel que soit le thème.

Je m'occuperai de la doc à ce sujet aussi, ça devrait être assez rapide :)

On 9 juillet 2014 21:05:32 HAEC, Alexandre Alapetite notifications@github.com wrote:

@GLLM Semble bien parti 👍 N'hésitez-pas à poster des captures
d'écran de la progression.

@aledeg ou quelqu'un d'autre : Ça pourrait être bien avec une petite
phrase au sujet du développement des thèmes dans la doc, si ça n'y est
pas déjà.


Reply to this email directly or view it on GitHub:
#533 (comment)

@marienfressinaud
Copy link
Member

@GLLM > pour répondre à tes questions, je viens de mettre en ligne un nouveau dépôt contenant un thème de base : https://github.com/marienfressinaud/FreshRSS-base-theme

Tu trouveras donc dedans :

  • Un fichier metadata.json qui indique quelques infos utiles à FreshRSS
  • Un fichier template.css qui doit être la base de tous les thèmes. Il met en place la structure de chacun des éléments et permet d'avoir quelque chose "qui tient debout"
  • Un fichier base.css : celui-ci est facultatif (pas obligé de te baser dessus) mais permet de mieux agencer les différents blocs afin d'avoir une vue homogène. Je te conseille de l'utiliser.

Basiquement j'ai fait un copié-collé du thème Origine et j'ai viré les propriétés suivantes du fichier base.css : color, background*, border*, *shadow. Donc tu n'as plus qu'à rajouter ces propriétés "customisées" pour ton thème dans ce même fichier. C'est comme ça que j'ai procédé pour les thèmes Dark et Flat dans la branche dev et ça a plutôt bien fonctionné :)

Par contre ! Il va y avoir (un jour) de la casse du côté de la gestion des catégories / flux, ce qui posera soucis sans doute aux thèmes existants. C'est le but du fichier template.css que de limiter cette casse, donc il ne faut pas le modifier (ou alors remonter les modifs upstream). Ça veut dire que les modifications structurelles peuvent potentiellement tout casser, mais ça restera limité si tu mets le fichier template.css à jour :)

Et je n'ai pas compris ta question à propos du content {}

@aledeg
Copy link
Member

aledeg commented Jul 10, 2014

Ne faudrait il pas intégrer le thème de base dans le code de FreshRSS?

@Alkarex
Copy link
Member

Alkarex commented Jul 10, 2014

Ça pourrait être un thème "caché", par exemple avec un nom de dossier
commençant par un point.

@aledeg
Copy link
Member

aledeg commented Jul 10, 2014

+1

@marienfressinaud
Copy link
Member

Sans le "cacher" avec un point, simplement ne pas remplir le champ name du fichier metadata.json et cacher les thèmes qui n'ont pas de nom dans le champ de sélection. Mais oui bonne idée :)

@GLLM
Copy link
Author

GLLM commented Jul 11, 2014

SAlut @marienfressinaud salut tout le monde,

merci des réponses.
Pour la partie .content{}, je voulais juste dire qu'un smartphone & un PC fixe vont tous les deux utliser cette classe pour le style et donc hériter des mêmes margin (30px * 2 c'est pas pareil sur un pc et sur un smartphone :) ).

L'utilisation de @media et de min-width sur la taille de l'écran pourrait peut-être aider sur ce point.

Est-ce plus clair ?

Merci
GLLM

@marienfressinaud
Copy link
Member

Ok, donc tu voudrais que l'on réduise la taille des margins des listes dans les versions mobiles ? Parce que sinon à part ça je ne vois pas trop ce que l'on pourrait changer ^^

@GLLM
Copy link
Author

GLLM commented Jul 15, 2014

C'est une suggestion, qu'on puisse gérer les 2 (mobile device & PC) différement ;)

a+

@Alkarex
Copy link
Member

Alkarex commented Jul 17, 2014

@GLLM : Est-ce que le bug suivant correspond à ce que vous demandez ? #264

@GLLM
Copy link
Author

GLLM commented Jul 20, 2014

Bonjour @Alkarex

Oui, c'est l'idée. Avec comme suggestion que la base des 2 thèmes soit la même afin qu'il ne faille pas maintenir 2 thèmes pour un seul & même thème qui soit utilisable tant sur mobile que sur PC !!

Merci
GLLM

@marienfressinaud
Copy link
Member

@GLLM > Actuellement pour chaque thème, que ce soit pc ou mobile, tout est géré par les mêmes fichiers. Mais si tu vas voir à la fin d'un fichier tu verras que nous utilisons déjà les media-queries pour gérer l'affichage différemment sur mobile. Du coup je ne vois pas trop ce que tu suggères de plus ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants