Task 3 react api #3
Conversation
KseniyaYatskevich
left a comment
KseniyaYatskevich
left a comment
There was a problem hiding this comment.
Использование кнопки Search для перехода на другую страницу - не лучшее решение, не понятно для пользователя
| onChangeSize('5'); | ||
| setSize('5'); | ||
| } | ||
| }; |
There was a problem hiding this comment.
в данном случает также можно было бы использоать конструкцию switch
src/components/Articles.tsx
Outdated
| <select name="pageSize" value={size} onChange={handleChoose} onBlur={handleChoose}> | ||
| <option value="5">5</option> | ||
| <option value="10">10</option> | ||
| <option value="20">20</option> |
There was a problem hiding this comment.
Для option лучше создать массив значений и пройтити по этому массиву - проще измеить значения, добавить новые или удалить const pageSize = [ 5, 10, 20]
src/index.tsx
Outdated
| import App from './App'; | ||
|
|
||
| ReactDOM.render( | ||
| <BrowserRouter> |
There was a problem hiding this comment.
Не совсем понимаю зачем используется Router в этом задании...
src/pages/Dashboard.tsx
Outdated
| import { ChangeEvent, FC, useState } from 'react'; | ||
| import Articles from '../components/Articles'; | ||
| import axios from '../services/api'; | ||
| import { Article, Get200Articles, SortType } from '../types'; |
There was a problem hiding this comment.
Хорошая идея вынести типы в отдельный файл! Обычно название такого файла связано с названием файла к которому относится ComponentModel.ts
| const [page, setPage] = useState<number>(1); | ||
| const [pageSize, setPageSize] = useState<string>('5'); | ||
| const [totalPages, setTotalPages] = useState<number>(1); | ||
|
|
There was a problem hiding this comment.
Молодец, что используешь хуки
src/pages/Dashboard.tsx
Outdated
| checked={sortBy === SortType.publishedAt} | ||
| onChange={() => setSortBy(SortType.publishedAt)} | ||
| />{' '} | ||
| publishedAt |
There was a problem hiding this comment.
Для пользователя лучше использовать что-то более понятное, хотя бы published
There was a problem hiding this comment.
Заменено на publication date
2
3. Выполнено задание согласно пунктам: строка поиска, список резульатов, переключатели сортировки, пагинация