-
Notifications
You must be signed in to change notification settings - Fork 3
Fix #193 - Responsive version Header, Login and signup fixes #212
Conversation
Travis automatic deployment: Storybook book automatic deployment: |
src/components/hoc/OpenHoc.tsx
Outdated
@@ -19,7 +19,10 @@ export function openHoc<T>( | |||
{ open: false, visited: false }, | |||
{ | |||
toggle: (state: OpenType) => () => ({ open: !state.open, visited: true }), | |||
clickAway: () => () => ({ open: false, visited: true }), | |||
clickAway: () => () => { | |||
console.log("On clickaway"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this required?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No hehehehe
<Typography variant="h4" color="primary" inline> | ||
{`${primaryTitle}\u00a0`} | ||
{`${primaryTitle} `} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it is better to use {primaryTitle}
?
{phone => ( | ||
<Block className={classes.container} padding={phone ? "lg" : "xxl"} margin="xxl" align="right"> | ||
<Typography variant="subtitle1" inline> | ||
{"Don't have an IOV wallet?\u00a0"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do you think about using
instead of \u00a0
i think it is more easy to read and will allow you to use it directly in code without curly braces and variable substitution.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything is good. Please check small notices. Maybe I am wrong with them.
bfd7bb1
to
051c16e
Compare
Travis automatic deployment: Storybook book automatic deployment: |
Description
This PR implements mostly all bugs referenced in #193, basically, it solves bugs reported in Header's Menu, login and signup views, for making them more responsive in phones.
Also, by reference, it closes #193 #176 #175 #174 #180 #173
Basically, it solves: