-
Notifications
You must be signed in to change notification settings - Fork 1
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/ue design #23
base: dev
Are you sure you want to change the base?
Fix/ue design #23
Conversation
@@ -2,7 +2,7 @@ | |||
// For more information, check the common.json.ts file | |||
|
|||
export default { | |||
'browser': 'Guide des UEs', | |||
browser: 'Guide des UEs', |
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.
tu veux pas garder les ' ?
semester: comment.semester.code, | ||
date: comment.createdAt.toLocaleDateString(), | ||
})} | ||
{t('ues:detailed.comments.resume', { |
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.
summary ça serait mieux je pense plutôt que resume, ça peut faire penser que c'est "reprendre" ou "CV", et en plus en anglais ya des accents sur les "e", donc encore moins intuitif ^^
</p> | ||
)} | ||
<div className={styles.meta}> | ||
{!comment.isAnonymous && ( |
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.
<div> | ||
<div>{t('ues:detailed.comments.writtenDate', { date: comment.createdAt.toLocaleDateString() })}</div> | ||
{comment.updatedAt && ( | ||
<div>{t('ues:detailed.comments.updatedAt', { date: comment.updatedAt.toLocaleDateString() })}</div> |
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.
une icone clock pour chacune des 2 ça serait mieux je pense, ça fait bizarre de pas avoir d'icone à côté de updatedAt
t('ues:detailed.noWorkingTimeInfo') | ||
)} | ||
</div> | ||
</div> | ||
<div className={styles.divider} /> | ||
{!isAnnalUploaderOpen ? ( |
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.
c'est giga long comme ternaire, ça serait cool de la diviser, limite dans le même fichier si tu veux mais là c'est dur de retrouver les séparations de la ternaire (début, milieu et fin)
No description provided.