-
Notifications
You must be signed in to change notification settings - Fork 1
Реализация кнопки "Копировать ID" в админке #97
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
Conversation
…овал через новую функцию
…зрыв через gap, убрал прописывание ошибки в консоль, облегчил toast
src/components/TheReviewCard.vue
Outdated
| <v-card width="200"> | ||
| <template #text> | ||
| <v-btn class="w-100" color="red" text="Удалить" @click.stop="deleteComment" /> | ||
| <v-card-text class="d-flex flex-column" style="gap: 12px"> |
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.
В целом, почти идеально.
Если смотреть в доку, то кнопки в v-card заводятся через v-card-actions (ты используешь v-card-text) + если мы используем такой синтаксис, то нам уже не нужен template #text, поскольку он суть дублирует то же самое. На мой взгляд тут правильно будет сделать вот так:
<v-card-actions class="d-flex flex-column" style="gap: 8px">
<v-btn block variant="elevated" text="Копировать ID" @click.stop="copyCommentID" />
<v-btn block color="red" variant="elevated" text="Удалить" @click.stop="deleteComment" />
</v-card-actions>
</v-card>
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.
8px или 12 тут уже сути особо не имеет. Мне просто визуально приятнее с 8.
Вообще, наши дизайнеры в фигме декларируют так:
Все элементы дизайна должны быть кратны четырем: 4, 8, 12, 16, 20... Это необходимо для того, чтобы сохранять консистентность дизайна, было проще ориентироваться в выставлении числовых показателей при сборке интерфейсов, и для адаптивности дизайна на стороне фронтенда
Dmatrushka19
left a comment
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.
Все гуд. Мержим в main
Изменения
Детали реализации
Check-List
blackиisortдля Back-End илиPrettierдля Front-End?