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

Améliore la partie "Envie de s'investir ?" de la bibliothèque #4489

Merged
merged 2 commits into from
Aug 7, 2017

Conversation

Situphen
Copy link
Member

@Situphen Situphen commented Aug 7, 2017

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) Forum, mon post et celui d'Abdelazer

Améliore la partie "Envie de s'investir ?" de la bibliothèque

Capture d'écran

@Situphen Situphen added the C-Front Concerne l'interface du site label Aug 7, 2017
@motet-a
Copy link
Contributor

motet-a commented Aug 7, 2017

N’y aurait-il pas un peu trop de points d’exclamation dans les titres (je chipote) ?

@coveralls
Copy link

coveralls commented Aug 7, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling f513fe7 on Situphen:ameliore-envie-de-s-investir into 482960a on zestedesavoir:dev.

@Situphen
Copy link
Member Author

Situphen commented Aug 7, 2017

@motet-a : Peut-être ouais. En fait j'ai hésité et je trouve que ça fait trop neutre/pas assez enthousiaste sans les points d'exclamation. À voir, moi ça me dérange pas de changer pour ne pas en mettre hein.

@gllmc
Copy link
Member

gllmc commented Aug 7, 2017

Je suis d'accord, je pense qu'on peut enlever les points d'exclamation. Par ailleurs, il y a deux titres à l'impératif et un à l'infinitif, il faudrait choisir entre les deux. :)

@Situphen
Copy link
Member Author

Situphen commented Aug 7, 2017

@gcodeur Je modifie pour les exclamations. Pour l'infinitif c'est un oubli :)

@Situphen Situphen force-pushed the ameliore-envie-de-s-investir branch from f513fe7 to 4a99dd1 Compare August 7, 2017 07:58
@coveralls
Copy link

coveralls commented Aug 7, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 4a99dd1 on Situphen:ameliore-envie-de-s-investir into 482960a on zestedesavoir:dev.

@abdelaz3r
Copy link
Contributor

Ouais ça rend super !

@coveralls
Copy link

coveralls commented Aug 7, 2017

Coverage Status

Coverage remained the same at 88.878% when pulling 9a3fd4a on Situphen:ameliore-envie-de-s-investir into 685568f on zestedesavoir:dev.

@pierre-24
Copy link
Member

ça me semble bien aussi :)

@pierre-24 pierre-24 merged commit 0470719 into zestedesavoir:dev Aug 7, 2017
@pierre-24 pierre-24 added this to the Version de développement milestone Aug 7, 2017
@vhf vhf changed the title [V25] Améliore la partie "Envie de s'investir ?" de la bibliothèque Améliore la partie "Envie de s'investir ?" de la bibliothèque Aug 20, 2017
@Situphen Situphen deleted the ameliore-envie-de-s-investir branch August 11, 2018 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants