Skip to content

Instantly share code, notes, and snippets.

@kaineer

kaineer/0205.md

Last active February 5, 2025 08:17
Show Gist options
  • Save kaineer/0b24c06a05130c2bfac53a88051ac437 to your computer and use it in GitHub Desktop.
Save kaineer/0b24c06a05130c2bfac53a88051ac437 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
0205.md

Что не так

  • Общее замечание: надо привыкать раскладывать компоненты по отдельным каталогам и файлам

components/components.jsx

  • const body

    • НЕ НАДО называть компоненты именами существующих тегов. Именно для этого их обычно называют с БОЛЬШОЙ буквы
    • Тег body как правило НЕ заменяют компонентом, просто потому, что первый же компонент находится внутри body

  • const Semaphore

    • Пока норм, но надо добавить еще компонентов внутрь

  • const ColorBox

    • Пока нормально, но нужно, чтобы можно было передавать color и указывать backgroundColor

  • const Selector

    • Пока ни о чем вообще
    • Нет разметки
    • Нет переданных параметров
    • На компонент не похож
    • Непонятно, почему он используется в App.jsx, тогда как он должен быть использован прямо из Semaphore
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment