Open Glajik opened 3 years ago
title
в компоненте TopBottomBarsLayout
- ожидается, что туда можно передать строку, однако это на самом деле это id для FormattedMessage
. Лучше логику мультиязычного интерфейса вынести на уровень выше, чтобы в св-во попадала уже готовая строка.settingsAction
в компоненте TopBottomBarsLayout
пробрасывается в TopBar
и на его основе выполняется условный рендеринг иконки. Это не хорошо. Лучше пробросить компонент целиком, уже вместе со сконфигурированными обработчиками. Тогда TopBar
и TopBottomBarsLayout
будут довольно простыми и гибкими в использовании, т.к. в них не будет никакой логики. А иконки и их обработчики будут настраиваться в компоненте страницы.Main
можно будет просто сделать () => props.history.push('/main')
P.S. Я слил ветку feature/navbar
в develop
. Попробуй внести изменения в feature/navbar
и по окончании исправлений сделать pull request, указав develop
в качестве базовой. Это для более удобного ревью, просто чтобы я мог сосредоточиться на изменениях.
Сделать макет экрана с заголовком и панелью навигации.