vhrechukha / react_internship_2021 Goto Github PK

Обычно модалка должна закрываться когда ты кликаешь мимо неё. То есть если кликнуть по фото или тексту она закрываться не должна

Время редко когда показывают по UTC, переведи время в таймзону пользователя и сделай его более читабельным. Формат времени в разных странах тоже отличается, и для него тоже можно сделать локализацию. Для этого советую посмотреть как i18next работает вместе с библиотекой moment

Функции в этом файле это HOC лучше вынести их в отдельную папку src/hoc
И не стоит копипастить код, подумаю как можно сделать isAuth более универсальным

 if (photo != null) setPhotoState(photo);

что будет если photo === undefined или пустой строке?
Лучше провеять подобные условия следующим образом

if (photo) setPhotoState(photo);

Сделай ссылку кликабельной, а так же было бы неплохо добавить кнопку copy и при нажатии на неё ссылка сохранялась в буфер обмена

Это не очень красиво

Строки вроде 'light' и 'dark' лучше вынести в константы, тогда сложнее будет написать их неправильно

Думаю от этого можно избавиться, тогда выглядеть будет приятнее

из-за catalog_management появляется вот такая ошибка

Почитав больше про этот разрешение так и не понял зачем оно нужно в приложении. Без него отлично работает

Хотя бы для названий полей в модалке