-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: update design of the accounting page #209
Conversation
vojtechsimetka
commented
Oct 4, 2021
•
edited
Loading
edited
9e91c9b
to
883b650
Compare
86b1bf1
to
1bcac76
Compare
1363275
to
03497f4
Compare
e145f0c
to
3be5097
Compare
3be5097
to
c835f8c
Compare
} | ||
|
||
return ( | ||
<div className={`${classes.root} ${rootLevelClass}`}> | ||
<ListItem button onClick={handleClick} className={classes.header}> | ||
<ListItemText primary={<Typography variant={typographyVariant}>{label}</Typography>} /> | ||
{open ? <ExpandLess /> : <ExpandMore />} | ||
<div style={{ display: 'flex' }}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<div style={{ display: 'flex' }}>
is used 8 times in the project - I feel like this could be extracted as a super small helper component to avoid inline styles 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created an issue out of it #213