Skip to content

implement hometask7#72

Open
MarinaBarinova wants to merge 1 commit intoromabelka:masterfrom
MarinaBarinova:hometask7
Open

implement hometask7#72
MarinaBarinova wants to merge 1 commit intoromabelka:masterfrom
MarinaBarinova:hometask7

Conversation

@MarinaBarinova
Copy link

@MarinaBarinova MarinaBarinova commented Jul 28, 2018

  1. При обновлении страницы, локализация сбрасывается. Нужно сохранять lang в redux store? Как это сделать? App завернуть в connect?
  2. Вопрос про локализацию меню
    Вот так у меня не сработало:
<Menu>
  <MenuItem to="/articles">{translate('menu.articles')}</MenuItem>
  <MenuItem to="/comments">{translate('menu.comments')}</MenuItem>
  <MenuItem to="/filters">{translate('menu.filters')}</MenuItem>
  <MenuItem to="/counter">{translate('menu.counter')}</MenuItem>
</Menu>

Пришлось MenuItem оборачивать в декоратор и из app пробрасывать ключи словаря, т.е. так:

<Menu>
  <MenuItem to="/articles">menu.articles</MenuItem>
  <MenuItem to="/comments">menu.comments</MenuItem>
  <MenuItem to="/filters">menu.filters</MenuItem>
  <MenuItem to="/counter">menu.counter</MenuItem>
</Menu> 

Мне решение не нравится, т.к. в MenuItem пришлось обернуть children в функцию локализации translate, что ограничивает использование children только в качестве текста. Как сделать более универсально?

@romabelka
Copy link
Owner

Не работает, поскольку ты пытаешься в декораторе использовать конекст(Consumer) выше провайдера. А в целом все ок

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants