Ts react todo list hw#6#51
Conversation
To do list hw#4 5
| const App: React.FC = () => { | ||
| return ( | ||
| <> | ||
| <Header name={undefined} /> |
There was a problem hiding this comment.
почему явно сюда прокидываем undefined. если компонент используется где то еще, то name задаем дефолтное значение внутри компонента
| <Route path='/todos'> | ||
| <HomePage /> | ||
| </Route> | ||
|
|
| // ) | ||
| // } | ||
|
|
||
| // export default TodoItem |
There was a problem hiding this comment.
комментированный код удаляться должен
| return ( | ||
| <div className={s.wrap}> | ||
| <span className={s.todos}> | ||
| Active:{' '} |
There was a problem hiding this comment.
лучше задачать на этом уровне в s.numberTodos стиль маржин лефт если надо пробел
| margin-top: 0; | ||
| margin-bottom: 0; | ||
| list-style: none; | ||
| } */ |
| import UserMenu from '../UserMenu/UserMenu' | ||
| import { connect } from 'react-redux' | ||
|
|
||
| const Header = (props: { name: any }) => { |
There was a problem hiding this comment.
any тип не приемлим. по дефолту пустая строка
| <Navigation name={''} /> | ||
| {!props.name ? ( | ||
| <Link to='/login'> | ||
| <BigButton text='Log In' icon={''} disabled={false} /> |
There was a problem hiding this comment.
дефолтное значение прокинуть в компонент, тут не вызывать, делаем его не обязательным если это используется где то еще
| <img className={s.logo} src={logo} alt='logo' /> | ||
| <div className={s.logoName}>Todo List</div> | ||
| </div> | ||
| <Navigation name={''} /> |
There was a problem hiding this comment.
дефолтное значение прокинуть в компонент, тут не вызывать, делаем его не обязательным если это используется где то еще
| <BigButton text='Log In' icon={''} disabled={false} /> | ||
| </Link> | ||
| ) : ( | ||
| <UserMenu name={''} todos={[]} /> |
There was a problem hiding this comment.
дефолтное значение прокинуть в компонент, тут не вызывать, делаем его не обязательным если это используется где то еще
2 participants
No description provided.