Пути */lessons/:id. Отсюда становится понятно, что мы просматриваем набор уроков, хотя должны брать отдельный урок. То есть, исходя из логики, нужно превратить этот путь в */lesson/:id
Пути _components/Lesson. Тут претензия в том, расположение компонентов не совпадает с существующей на проекте иерархией файлов. По идее, к компонентам урокам должен быь путь _components/platform/Lesson
Именование файла MainImage.tsx некорректно, т.к. у тебя уже есть компонент Image и в его окружении не может быть MainImage, BasicImage и тому подобного. Предвкушая вопрос, а почему? Много причин. Из основного: рушится устоявшаяся иерархия, где такие моменты решаются прокидыванием пропсов типа variant, разрастается объем кода и файлов.
@Fealerok
Конкретно претензия к трем следующим моментам:
*/lessons/:id
. Отсюда становится понятно, что мы просматриваем набор уроков, хотя должны брать отдельный урок. То есть, исходя из логики, нужно превратить этот путь в*/lesson/:id
_components/Lesson
. Тут претензия в том, расположение компонентов не совпадает с существующей на проекте иерархией файлов. По идее, к компонентам урокам должен быь путь_components/platform/Lesson
MainImage.tsx
некорректно, т.к. у тебя уже есть компонентImage
и в его окружении не может бытьMainImage
,BasicImage
и тому подобного. Предвкушая вопрос, а почему? Много причин. Из основного: рушится устоявшаяся иерархия, где такие моменты решаются прокидыванием пропсов типа variant, разрастается объем кода и файлов.