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

Re-design de la sélection des thèmes #571

Closed
Alwaysin opened this issue Aug 8, 2014 · 11 comments
Closed

Re-design de la sélection des thèmes #571

Alwaysin opened this issue Aug 8, 2014 · 11 comments
Labels
Milestone

Comments

@Alwaysin
Copy link
Contributor

Alwaysin commented Aug 8, 2014

Bonsoir,

En marge de la proposition d'Alkarex sur #557 (qui serait plutôt un site externe présentant les thèmes, de ce que j'ai compris), il serait intéressant d'avoir un visuel lors du choix d'un thème.
Je joins un montage réalisé très rapidement (les noms des thèmes sont les mêmes, les captures sont moches, etc, mais c'est pour montrer l'idée).

2014-08-08 23_11_14-affichage freshrss

Et là on voit par exemple que j'aurais sélectionné le premier thème ^^
Qu'en pensez-vous ?

@marienfressinaud
Copy link
Member

Tout à fait ! Très bonne idée :) Par contre il faudra voir à limiter cette zone (avec possibilité de l'étendre) afin que ça ne bouffe pas toute la place à l'écran avec beaucoup de thèmes.

marienfressinaud added a commit that referenced this issue Aug 9, 2014
- Not include screenshot yet (wait
  #571)
- Need maybe more tests?

See #557
@poulpito
Copy link

Pourquoi pas en miniature Rollover
Genre png 32*32 inclu dans le thème en guise de selection + icone pour le courant
et une preview plus grande en rollover
Ca éviterai bien de se faire flooder 3 pages de thèmes

@marienfressinaud
Copy link
Member

Le problème des rollovers c'est que je n'en ai jamais trouvé un facile à utiliser. On n'arrive jamais vraiment à viser comme on veut. De plus on ne peut pas avoir de vue d'ensemble ce que je trouve désagréable ;)

Je trouverai une solution pas de soucis, et de toutes façons je ne compte pas ajouter 40 000 thèmes différents dans le package de base. Une petite dizaine sera bien le maximum !

@marienfressinaud marienfressinaud added this to the 0.9.0 milestone Aug 12, 2014
aledeg added a commit that referenced this issue Nov 15, 2014
I made a CSS3 gallery to select the theme. Now you can see the theme before applying it.
Comments are welcome.

See #571
aledeg added a commit that referenced this issue Nov 15, 2014
I made a CSS3 gallery to select the theme. Now you can see the theme before applying it.
Comments are welcome.

See #571
@Alwaysin
Copy link
Contributor Author

Réglé avec ce qu'a fait @aledeg je pense :)
J'aime beaucoup !!!

@aledeg
Copy link
Member

aledeg commented Nov 16, 2014

Merci

@Alwaysin
Copy link
Contributor Author

Je pense qu'il faut mettre un délimiteur ou autre, car quand la description est trop longue cela dépasse sur la capture et c'est illisible :

2014-11-24 17_38_57-affichage freshrss

@Alwaysin Alwaysin reopened this Nov 24, 2014
@marienfressinaud
Copy link
Member

Je voulais faire des retouches de design un peu plus tard ;) Merci à @aledeg d'avoir implémenté ça déjà !

@marienfressinaud
Copy link
Member

J'ai fait quelques retouches, notamment pour corriger le problème cité. C'est pas encore tout à fait comme je voudrais mais ça s'en rapproche.

@aledeg > pourquoi avoir utilisé la propriété user-select: none; ? Je ne vois pas l'intérêt, c'est même génant puisque ça empêche le copié-collé.

@aledeg
Copy link
Member

aledeg commented Dec 12, 2014

En fait, j'ai adapté un code trouvé. J'ai laissé cette partie car je ne voyais pas trop l'intérêt de pouvoir sélectionner le texte à cet endroit.

@marienfressinaud
Copy link
Member

Personnellement je trouve que ça pose plus de problèmes que ça n'apporte d'intérêt : il y a certainement plus de bonnes raisons pour sélectionner du texte que pour en interdire l'usage. Je supprime donc cette partie.

@marienfressinaud
Copy link
Member

Je ferme ce ticket pour le moment. S'il y a besoin d'améliorations on ouvrira un autre ticket ;)

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