Skip to content
Snippets Groups Projects
Commit 09d769aa authored by Bogdan Cochior's avatar Bogdan Cochior
Browse files

feat(review): limit 1 file per upload and disable textarea when uploading file

parent 315a634d
No related branches found
No related tags found
1 merge request!10Sprint #12
...@@ -93,12 +93,16 @@ const ReviewerReportForm = ({ ...@@ -93,12 +93,16 @@ const ReviewerReportForm = ({
<Row left> <Row left>
<Label>Report</Label> <Label>Report</Label>
{!fileFetching.review ? ( {!fileFetching.review ? (
<FilePicker <Fragment>
allowedFileExtensions={['pdf', 'doc', 'docx']} {isEmpty(formValues.files) && (
onUpload={addFile} <FilePicker
> allowedFileExtensions={['pdf', 'doc', 'docx']}
<ActionText left={12}>Upload file</ActionText> onUpload={addFile}
</FilePicker> >
<ActionText left={12}>Upload file</ActionText>
</FilePicker>
)}
</Fragment>
) : ( ) : (
<Spinner size={2} /> <Spinner size={2} />
)} )}
...@@ -111,10 +115,12 @@ const ReviewerReportForm = ({ ...@@ -111,10 +115,12 @@ const ReviewerReportForm = ({
{...input} {...input}
hasError={input.validationStatus === 'error'} hasError={input.validationStatus === 'error'}
onChange={e => changeField('public', e.target.value)} onChange={e => changeField('public', e.target.value)}
readOnly={fileFetching.review}
rows={6} rows={6}
/> />
)} )}
name="public" name="public"
readOnly={fileFetching.review}
validate={isEmpty(formValues.files) ? [required] : []} validate={isEmpty(formValues.files) ? [required] : []}
/> />
</FullWidth> </FullWidth>
...@@ -153,10 +159,12 @@ const ReviewerReportForm = ({ ...@@ -153,10 +159,12 @@ const ReviewerReportForm = ({
{...input} {...input}
hasError={input.validationStatus === 'error'} hasError={input.validationStatus === 'error'}
onChange={e => changeField('confidential', e.target.value)} onChange={e => changeField('confidential', e.target.value)}
readOnly={fileFetching.review}
rows={6} rows={6}
/> />
)} )}
name="confidential" name="confidential"
readOnly={fileFetching.review}
validate={[required]} validate={[required]}
/> />
</FullWidth> </FullWidth>
...@@ -311,6 +319,10 @@ const Textarea = styled.textarea` ...@@ -311,6 +319,10 @@ const Textarea = styled.textarea`
font-family: ${th('fontWriting')}; font-family: ${th('fontWriting')};
border-color: ${({ hasError }) => border-color: ${({ hasError }) =>
hasError ? th('colorError') : th('colorPrimary')}; hasError ? th('colorError') : th('colorPrimary')};
transition: all 300ms linear;
&:read-only {
background-color: ${th('colorBackgroundHue')};
}
` `
const Spacing = styled.div` const Spacing = styled.div`
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment