Skip to content

Conversation

@Xrias
Copy link
Collaborator

@Xrias Xrias commented May 18, 2018

А вот и ночная тема подъехала

А вот и ночная тема подъехала
Copy link
Collaborator

@Damirikys Damirikys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

После перезагрузки все валится: см. в консоль - ReferenceError: document is not defined
Из за метода isDeclarationSupported

Ну и иконку смены темы запилить бы чуть-чуть меньше и в уголок в правый + нужно подключить сюда redux и сделать так, чтобы состояние темы запоминалось и не сбрасывалось при перезагрузке страницы

@@ -1,7 +0,0 @@
PORT=8080
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем удалил отсюда все?

<p className="profile-card__body_bio">{user.bio}</p>
<p className="profile-card__body_email">{user.email}</p>
</div>
<ThemeSwitch></ThemeSwitch>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

просто <ThemeSwitch/>

}
}

ThemeSwitch.defaultProps = {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пропсы у нас по другому декларируются:

type Props = {
...
}

class ThemeSwitch extends Component<Props> {

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants