Skip to content

Hw 3#1

Merged
vellun merged 21 commits intomasterfrom
hw-3
Apr 7, 2025
Merged

Hw 3#1
vellun merged 21 commits intomasterfrom
hw-3

Conversation

@vellun
Copy link
Copy Markdown
Owner

@vellun vellun commented Mar 31, 2025

No description provided.

@nikolya98
Copy link
Copy Markdown

В целом неплохо вышло

из недостатков:

  1. проблемы с вёрсткой:
  • нет адаптива
  • нет ограничивающего центрирующего контейнера
  • карточки могут "слипнуться" на определённых разрешениях
  • не робит кнопка назад
  • на деталке белый фон содержимого не тянется на всю страницу, если данных мало
  • нет лоадеров
  • не используется БЭМ
  1. логику можно было лучше разделить, более декларативно

это можно сливать, правки со следующим дз сдашь, в новом pr + разделение вью и логики сразу на mobx делаем с использованием сторов

@vellun vellun merged commit 743c450 into master Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants