WIP: Dropzone #17
WIP: Dropzone #17
Conversation
topolanekmartin
left a comment
topolanekmartin
left a comment
There was a problem hiding this comment.
Příště bych zvolil vhodnější commit zprávu. Zpráva "commit" toho moc neřekne, když si budeš někdy zpětně procházet git historii :D Dospal jsem ještě pár bodů, které bych zlepšil :) jinak supr práce.
| @@ -0,0 +1,7 @@ | |||
| { | |||
There was a problem hiding this comment.
Smazal bych tento soubor. Už ho tu jednou máme jako .prettierrc.json
| StyledSVGCircleTwo, | ||
| StyledSVGCircleText, | ||
| } from './style/Circle.style'; | ||
| import { ColorResult } from 'react-color'; |
There was a problem hiding this comment.
Tento import jsi nevyužila, prosím smazat.
| dashOffset={currentDashOffset} | ||
| /> | ||
| </StyledSVG> | ||
| <StyledSVGCircleText key={progress} type={circleTwoType}> |
There was a problem hiding this comment.
Netuším proč tato komponenta má v názvu SVG. Zvolil bych spíš název StyledLoadingCircleText
| transform: rotate(-90deg); | ||
| transition: stroke 1s ease; | ||
|
|
||
|
|
There was a problem hiding this comment.
Zde ti vznikl řádek navíc. Smazat
| props.type === 'success' && | ||
| css` | ||
| stroke: ${props.theme.color.success.alpha}; | ||
| `} |
There was a problem hiding this comment.
Jednotlivé bloky bych odděloval řádkem. Blokem mám na mysli například toto
${(props) =>
props.type === 'success' &&
css`
stroke: ${props.theme.color.success.alpha};
`}
|
|
||
| &:hover { | ||
| border: 1px dashed ${(props) => props.theme.color.primary.alpha}; | ||
| ${StyledDropZoneText} { |
There was a problem hiding this comment.
Oddělit řádkem :)
| border: 1px dashed ${(props) => props.theme.color.border}; | ||
|
|
||
| &:hover { | ||
| border: 1px dashed ${(props) => props.theme.color.primary.alpha}; |
There was a problem hiding this comment.
je potřeba zde nastavovat opět stejné hodnoty, co jsi nastavila o tři řádky výš? Nemohli bychom v hoveru nastavit jen barvu borderu? :)
Je to i z toho důvodu, že kdybys náhodou chtěla změnit typ nebo šířku borderu tak to musíš upravovat na dvou místech.
| `; | ||
|
|
||
| export const StyledDropZoneIcon = styled(Icon)<StyledDropZoneIconProps>` | ||
|
|
There was a problem hiding this comment.
Smazat řádek, jinak supr komponenta :)
|
|
||
| export const StyledUploadingTitle = styled(Title)<StyledUploadingTitle>` | ||
| text-align: center; | ||
| ${(props) => |
There was a problem hiding this comment.
tyto dva bloky oddělit řádkem
| `} | ||
| `; | ||
|
|
||
| export const StyledProgressCircle = styled.div` |
There was a problem hiding this comment.
Od tohoto řádku a níž jsou komponenty, které máš definované jinde, nepletu se? Pokud jo, tak to smaž
2 participants
No description provided.