#107- Adaptive notifications made#116
Conversation
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
|
|
Мне немного не нравятся отступы. Между самими уведомлениями и лентой отступа вовсе нет и могут быть ложные нажатия либо на иконку настроек, либо на первое уведомление. Плюс, как будто на 25-50% ябы уменьшил отступы по краям 
|
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
| {isMobile ? ( | ||
| <Drawer open={isOpen} onOpenChange={handleOpenChange}> | ||
| <DrawerTrigger asChild> | ||
| <Button variant="none" className="relative h-[32px] w-[32px] p-1"> |
There was a problem hiding this comment.
Тут и не только лучше для размеров, кратных 4, использовать единицы tailwind, то есть h-8 w-8
| /> | ||
| </div> | ||
| ))} | ||
| {isMobile ? ( |
There was a problem hiding this comment.
Я бы быть может как-то иначе код организовал, чтобы разделение было через if (isMobile) return
Условные выражения с большим контентом сложно читаемы
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
|
Подумала, что лучше сделать в целом рефакторинг Notifications.tsx для упрощения файла, поэтому поделила на более мелкие компоненты и хук, перенесла весь функционал уведомлений в отдельную папку Header/Notifications. Так ок делать или лучше было оставить как есть и внести правки только там где надо было? Спасибо! |
There was a problem hiding this comment.
Вынеси, пожалуйста, хук в папку на уровне пакета, у нас всё же здесь ui лежит
There was a problem hiding this comment.
Да, сорри, протупила и свалила все в одну папку. Исправила !
xyzelena
temporarily deployed
to
xi-development
GitHub Actions
Inactive
2 participants
https://github.com/xi-effect/xi.progress/issues/107