poprawione other_buttons #40
Conversation
| import { faDownload} from '@fortawesome/free-solid-svg-icons'; | ||
| import { FontAwesomeIcon } from '@fortawesome/react-fontawesome'; | ||
|
|
||
|
|
There was a problem hiding this comment.
1 rzecz, to używaj prettier i eslinta, bo widzę, że nie ma tutaj odpowiednich wcięć, odstępów itp. Prettier podczas zapisywania posortuje Ci cały kod wedle schematu narzuconego w konfiguracji, która jest zapisana w projekcie
| variant: string; | ||
| }; | ||
|
|
||
| const OtherButton = ({variant}:Props) => { |
There was a problem hiding this comment.
Nazwa komponentu nie wskazuje na to co on faktycznie robi, a powinna sugerować to co jest wyświetlane. np. CircularButton -> Okrągły przycisk, a tutaj OtherButton nie wiele nam mówi
| return( | ||
| <div className='d-none d-sm-inline-block btn btn-sm btn-primary shadow-sm'> | ||
| <FontAwesomeIcon icon={faDownload} /> | ||
| Pobierz Konfigurację |
There was a problem hiding this comment.
Tekstu nigdy nie przypisujemy sztywno, zawsze musi to być tłumaczenie
|
Do pull requesta najlepiej dołączaj screena z tego co zrobiłeś, w tym przypadku screena z tego jak wygląda komponent w aplikacji. Wówczas osoba robiąca review ma dużo prościej zwizualizować sobie efekty |
No description provided.