-
-
Notifications
You must be signed in to change notification settings - Fork 76
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
Epic: Review file tree and update MUI Components #1617
Comments
#1549 has a list of all the files with jsx
|
Hey all, @chukalicious was asking what we should do for components that aren't being used and aren't accessible if they're in this MUI update list. We should close those issues as unplanned and mark in the list after the issue number that the component isn't used in the current iteration of the app. eg
And inside that issue mark it as unplanned |
@trillium Can you please close issues containing components that are not being used, add text: [ closed, unused component ] next to these issues, and make an update to this issue within the section "What if we're unable to access the component listed." |
@JackHaeg Done :) |
Overview
Currently, components throughout the VRMS app do not match - some use MUI styling while others do not, thus creating a fractured user experience. All components throughout the VRMS file tree below must be updated to Material-UI components.
What if we're unable to access the component listed
Do your best to change the HTML elements out anyway and note in the PR that you're unable to get the component to render within VRMS for screenshots.
Should I try to match the CSS of the app beforehand
No, there will be some changes to the look and feel of the app switching to MUI components, we plan on going back in a second set of issues to match the CSS to our current figma design or a new figma design as it comes available.
Action Items
Check off these items when we have fully moved from standard HTML components to MUI components (note that same may already be updated to MUI). Each can be broken into its own issue.
Resources/Instructions
https://mui.com/base-ui/all-components/
The text was updated successfully, but these errors were encountered: