-
Notifications
You must be signed in to change notification settings - Fork 161
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
Fix #1731: Ajout de la possibilité de trier ses articles/tutoriels #1733
Conversation
Je peux pinailler ?
|
Le filtre "par date de dernière modificiation" dans les articles/tutos ne marche que sur les articles en ligne c'est ca ? (car sur les brouillons ca pas l'air ca) Sinon c'est tout bon pour moi (aux remarques précédentes près) |
Je vais rajouter tout cela alors. Je vais vérifier le comportement pour les brouillons, ce n'est pas normal. |
@Eskimon ça vient d'un autre bug qui ne mets pas la date à jour je suppose. J'ai une PR en cours. Pour les icones vertes j'ai rajouté la classe mais il va falloir faire les icones « gear » en vert (ou en trouver une autre). |
Pourquoi pas utiliser les "ticks" bleu/vert comme sur les autres filtres ? |
Je trouve ça pas du tout adapté. |
On fait quoi ici ? |
@@ -37,6 +37,13 @@ | |||
{% elif request.GET.type == "draft" %} | |||
/ Brouillons | |||
{% endif %} | |||
{% if sort == "abc" %} | |||
/ Par ordre alphabétique |
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.
Ne serait-il pas mieux de "variabiliser" ces constantes?
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.
Aie en effet, des comparairons de texte comme ça ne devrait pas avoir lieu. Il faut définir une classe en amont type_sort
avec des conditions du style is_alpha
, is_creation
, etc.
Je changerai ça quand j'aurai un peu de temps. |
PR REPRISE DANS #1951
QA
Dans « Mes articles » :