Première tentative de templates #31
No reviewers
Labels
No labels
documentation
Doing
Doing
Good first issue
To Do
To Do
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: DGNum/authens#31
Loading…
Reference in a new issue
No description provided.
Delete branch "kerl/templates"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Pas mal de trucs piqués dans GestioCOF mais recodés sans bootstrap.
J'ai un peu changé les couleurs parce que je voulais que ce soit un peu différent quand même.
Avant de merge :
{% block %}à override ?changed milestone to %1
added 5 commits
masterCompare with previous version
Pas ultra fan de la police monospace, ça fait plus dev qu'autre chose :slight_smile:
"Authentification ENS", tant qu'à faire ?
Peut-être "AuthENS", pour garder le côté international...
Pour le reste, peut être que @champeno aura des meilleures remarques :slight_smile:
Oui, moi j'aime bien mais ça fait peut-être un peu nerd effectivement.
Je te propose :
sans-serifà la place (je ne veux pas mettre faire télécharger une police fancy aux gens vu que c'est une page de login "par défaut", ça pourra se customiser côté gestioCOF par exemple si on y tient)font-weight: bolddansa:hoverparce qu'avec une police non-monospace ça fait bouger le texte, j'ai peur que ce soit désagréableJ'ai une légère préférence pour un nom court pour que ça passe dans l'onglet du navigateur (c'est pas très important ceci dit)
J'ai appelé le projet "authens" mais "ensauth" serait plus correct en anglais. Une fois de plus ce n'est pas très grave. Mais ça ne me choquerait pas de renommer.
Une troisième option c'est de mettre juste "login"
Bref, j'ai proposé plus de trucs mais je n'aide pas à trancher :P Tu préfères quoi ?
sans-serifme paraît bien oui !Effectivement le fond est un peu tristoune, je coupe la 🍐 en deux et je mets #ddd ? ;)
Effectivement le violet ENS c'est pas une mauvaise idée mais je trouve qu'il ne va pas avec le vert à côté, faudrait retrouver d'autres couleurs pour les extés (et les vieux clippers)…
(et j'aimais bien mes couleurs)
added 11 commits
masterCompare with previous version
(finalement je mets ton #eee pour le fond, j'ai changé d'avis je préfère)
Je suppose que dans la majorité des cas ça sera la première option, donc ça n'est pas spécialement grave 😄
Pour rester cohérent,
AuthENS - Login?Va pour les couleurs originales, du coup =)
Quelque chose que j'avais oublié de mentionner : j'aime bcp l'idée de
{% if request.site.name %}{{ request.site.name }} - login{% else %}ENS Auth{% endif %}, du coup ça serait sûrement pas mal de la mettre dans le container aussi !deal
changed this line in version 4 of the diff
added 9 commits
master7d56fe15- Templates: first draft73e79d65- Apply some of Aufinal's design remarksa6f4bfcb- Better html titleCompare with previous version
resolved all discussions
C'est à dire ? Tu proposes que ça apparaisse sur la page ?
Sur cette ligne, mettre aussi un "AuthENS" (ou le nom du site le cas échéant), avant la partie "Mode de connexion".
added 1 commit
b0a0c488- A little bit styling for form inputsCompare with previous version
J'ai un peu amélioré les
inputs, qu'est ce que tu en penses ?Effectivement, c'est mieux !
added 1 commit
fa82ee25- Put the site's name in every h2Compare with previous version
J'arrive pas à comprendre ou pointe ton lien (à cause des mes force-push ?) mais je crois que j'ai compris : cf
fa82ee25.Tu voulais dire ça ?
Si oui, je trouve que c'est une bonne idée
Oui, c'était bien ça =)
Après test, j'ai l'impression que les erreurs dans les formulaires ne sont pas très bien supportées =)
ah oui, c'est même carrément moche effectivement, j'avais pas testé.
added 1 commit
bc1632dd- Errors deserve CSS too!Compare with previous version
ça devrait être mieux maintenant
C'est presque parfait ! Je rajouterais peut-être un peu de styling sur le bouton de login, mais si tu veux laisser tel quel ça se merge aussi =)
hum, tu penses à quoi ? J'ai laissé le styling de base même si c'est pas le plus beau du monde pour que ça reste bien identifiable comme étant un bouton (avec les bordures rondes), mais je suis preneur si tu as une idée.
Et j'avais pas capté mais c'est en anglais alors que le reste est en français donc ça au moins je change
Principalement plus gros je dirais, et peut être changer la couleur de fond pour aller avec le colorscheme (vert plus foncé ?)
Pour ce qui est de l'anglais, c'est un problème à un autre endroit que je ne sais pas comment régler : si tu laisses un champ vide dans la form de login, l'erreur dit "Please fill out this field"...
Bizarre, moi j'ai "veuillez compléter ce champ".
Je pense que c'est une feature de HTML, c'est indépendant de Django : quand tu mets
required=""dans les attributs de l'input ton navigateur doit te mettre le message. Mon navigateur est en français et j'en déduis que le tiens est en anglais ?Hmm, pas con 😄
added 1 commit
6042a502- Fancier submit buttonCompare with previous version
Je trouvais que le vert foncé ne marchait pas trop, j'ai tenté le même gris que le header pour rester dans le thème, dis moi ce que tu en penses.
Je ne suis pas 100% convaincu par le
border-color: white;quand on:hovermais je voulais avoir un truc visuel quand même. Peut-être un léger changement de couleur sinon ?Ça me va bien avec le
border-color! Je merge ça comme ça, c'est largement utilisable ; de toute façon, ça se customise par projet ensuite si on veut fine-tuner des trucs =)resolved all discussions
merged
mentioned in commit
6e51e555f6👍
resolved all discussions