-
-
Notifications
You must be signed in to change notification settings - Fork 79
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
update userprofile to mui #1766
base: development
Are you sure you want to change the base?
update userprofile to mui #1766
Conversation
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.
Looks like a good PR to me, thanks for putting work into these changes!
Would be nice to check on seeing if Tpyography
element needs component='h3' and variant='h3' but not requried
<div className="profile__header"> | ||
<h3 className="profile__title">My Profile</h3> | ||
</div> | ||
<Box> |
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.
Eventually change hex code to a stored variable, but for now this fine
</div> | ||
<Box> | ||
<Box style={{ backgroundColor: '#bad3ff' }}> | ||
<Typography |
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.
Curious if one of these is redundant, I tought that Typography variant="h3" would make this component an h3 element already
<Typography | ||
variant="h3" | ||
component="h3" | ||
style={{ |
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.
We may be able to have this be declared through the MUI config file later on
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.
Looks like a good PR to me, thanks for putting work into these changes!
Would be nice to check on seeing if Tpyography
element needs component='h3' and variant='h3' but not requried
Fixes #1715
What changes did you make and why did you make them ?
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
Visuals before changes are applied
[image](
)
Visuals after changes are applied
[image](
)