search

yellowflx / word-game

Russian analog of Wordle game for learning purpose
https://yellowflx.github.io/word-game/
2 stars 1 forks source link

short review #1

Closed ds-pankov closed 2 years ago

ds-pankov commented 2 years ago
  1. Удали .idea и добавь в .gitignore.
  2. word-game/src/store/redusers ошибка в слове.
  3. word-game/src/store/redusers/board/actron-creators.ts => actionCreators.ts.
  4. Все файлы переименуй в camelCase формат.
  5. Хороший совет, файл с функцией называй по названию функции: например getGovneco.ts, processGovneco.ts, formatGovneco.ts.
  6. export const useTypedSelector: TypedUseSelectorHook<RootState> = useSelector – этому есть решение, погугли.
ds-pankov commented 2 years ago

@YellowFLX FYI

ds-pankov commented 2 years ago

https://github.com/YellowFLX/word-game/blob/e46ab02b90d4c2d7eeb74e7cef5320113e1cb8c0/src/components/LetterCell.tsx#L8

Переименуй LProp => LetterCellProps и во всех других местах тоже.

ds-pankov commented 2 years ago

Убери дефолтные экспорты, они сильно замедляют рефакторинг в будущем.

ds-pankov commented 2 years ago

Убери все стили компонентов из src/App.css.

Стили должны лежать рядом с компонентом, и использоваться css-module или styled-components, советую попробовать styled-components.

Структура компонента: