Skip to content

Commit 57c962d

Browse files
committed
Refactor AB#49: Enhanced reponsivity from ErrorResult component
1 parent f1f8e82 commit 57c962d

File tree

2 files changed

+56
-18
lines changed

2 files changed

+56
-18
lines changed

src/pages/Articles/ErrorResult.jsx

+20-18
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,17 @@
11
import React from 'react';
22
import PropTypes from 'prop-types';
33

4-
import {Box, Button, Icon, Typography} from '@material-ui/core';
4+
import {Box, Button, Typography} from '@material-ui/core';
55

6-
import {SampleContainer} from './styles';
6+
import {
7+
SampleContainer,
8+
ArticleErrorResultIconContainer,
9+
ArticleErrorResultIcon,
10+
ArticleErrorResultMessageContainer,
11+
} from './styles';
712

813
const ErrorResult = (props) => {
9-
const {visible} = props;
14+
const {visible, message} = props;
1015

1116
return (
1217
<SampleContainer item xs={12} visible={visible.toString()}>
@@ -25,20 +30,14 @@ const ErrorResult = (props) => {
2530
m={2}
2631
>
2732
<Box display="flex" alignItems="center" flexWrap="wrap">
28-
<Box
29-
display="flex"
30-
justifyContent="center"
31-
alignItems="center"
32-
className="error-icon-area"
33-
>
34-
<Icon color="primary" className="error-icon">healing</Icon>
35-
</Box>
36-
<Box display="flex" justifyContent="center" alignItems="center">
37-
<Typography component="h2" variant="h4">
38-
Ops! ocorreu um erro ao buscar nossos artigos.
39-
Já tentou atualizar a página?
33+
<ArticleErrorResultIconContainer>
34+
<ArticleErrorResultIcon color="primary">healing</ArticleErrorResultIcon>
35+
</ArticleErrorResultIconContainer>
36+
<ArticleErrorResultMessageContainer>
37+
<Typography component="h2" variant="h5">
38+
{message}
4039
</Typography>
41-
</Box>
40+
</ArticleErrorResultMessageContainer>
4241
</Box>
4342
<Box display="flex" flexDirection="column" width="100%" mt={3}>
4443
<Button
@@ -57,7 +56,7 @@ const ErrorResult = (props) => {
5756
variant="contained"
5857
onClick={() => window.location.href = '/sobre#contact'}
5958
>
60-
Reportar bug
59+
Fale conosco
6160
</Button>
6261
</Box>
6362
</Box>
@@ -68,10 +67,13 @@ const ErrorResult = (props) => {
6867

6968
ErrorResult.propTypes = {
7069
visible: PropTypes.bool,
70+
message: PropTypes.string,
7171
};
7272

73-
ErrorResult.defaultProp = {
73+
ErrorResult.defaultProps = {
7474
visible: false,
75+
message: `Ops! ocorreu um erro ao buscar nossos artigos.
76+
Já tentou atualizar a página?`,
7577
};
7678

7779
export default ErrorResult;

src/pages/Articles/styles/index.js

+36
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,13 @@ export const ArticleTextContentPlaceholder = styled(Box)({
165165
minHeight: '600px',
166166
});
167167

168+
export const ArticleTextContentErrorResultContainer = styled(Box)({
169+
minHeight: '650px',
170+
display: 'flex',
171+
alignItems: 'center',
172+
justifyContent: 'center',
173+
});
174+
168175
export const ArticleFooterContainer = styled(Grid)({
169176
display: 'flex',
170177
alignItems: 'center',
@@ -179,3 +186,32 @@ export const ArticleFooterIconButton = styled(FontAwesomeIcon)({
179186
color: (props) => props.state === 'disabled' ? COLORS.disabled : COLORS.primaryHovered,
180187
},
181188
});
189+
190+
export const ArticleErrorResultIconContainer = styled(Box)({
191+
display: 'flex',
192+
justifyContent: 'center',
193+
alignItems: 'center',
194+
margin: 4,
195+
fontSize: '3rem',
196+
[devices.laptop]: {
197+
width: '100%',
198+
marginTop: 25,
199+
marginBottom: 25,
200+
},
201+
});
202+
203+
export const ArticleErrorResultIcon = styled(NoResultIcon)({
204+
fontSize: '3rem',
205+
});
206+
207+
export const ArticleErrorResultMessageContainer = styled(Box)({
208+
display: 'flex',
209+
justifyContent: 'center',
210+
alignItems: 'center',
211+
textAlign: 'center',
212+
[devices.laptop]: {
213+
width: '100%',
214+
marginTop: 20,
215+
marginBottom: 20,
216+
},
217+
});

0 commit comments

Comments
 (0)