задачи MeetupView и UiClock #8
Conversation
|
Добавляю преподавателя (@ShGKme) для код-ревью. |
|
Решение было обновлено, посмотрим что скажет @ShGKme |
| const date = new Date(); | ||
| const options = { timeStyle: 'medium' } | ||
|
|
||
| formattedTime.value = new Intl.DateTimeFormat('en-US', options).format(date); |
There was a problem hiding this comment.
Как и в прошло теме, вместо фиксированной локализации (en-US или ru-RU) лучше всегда использовать navigator.language для использования локализации пользователя.
| @@ -15,10 +15,13 @@ export default defineComponent({ | |||
| }, | |||
| }, | |||
|
|
|||
| emits: ['button-clicked'], | |||
There was a problem hiding this comment.
События обычно называют как название действия (существительное), а не причастие. button-click. Аналогично событиям в браузере - нативное событие click, а не clicked.
There was a problem hiding this comment.
В целом в имени события лучше описывать не действия пользователя, а происходящее действие. В этом компоненте важно, не что "пользователь нажал на какую-то кнопку" (какую? а если их несколько? а если удалить можно без кнопки?), а удаление email-а.
Лучше подойдут названия remove, delete
No description provided.