Skip to content
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

Updated HTML to MUI components on index.jsx #1762

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

nora-zajzon
Copy link
Member

@nora-zajzon nora-zajzon commented Sep 25, 2024

Fixes #1679

What changes did you make and why did you make them?

  • Changed all the HTML components to MUI applicable components
  • Used inline styling

Screenshots of Proposed Changes to the Website (if any)

Visuals before changes were applied Screenshot 2024-09-24 at 8 27 20 PM
Visuals after changes were applied Screenshot 2024-09-25 at 3 19 53 PM

@nora-zajzon nora-zajzon changed the title updated to mui using new branch due to complications on other pr Updated HTML to MUI components on index.jsx Sep 25, 2024
@angela-lee1 angela-lee1 self-requested a review October 8, 2024 14:07
@angela-lee1
Copy link
Member

angela-lee1 commented Oct 8, 2024

I pulled down the changes to my local environment and verified that all HTML components have been updated to MUI. The visuals match the screenshot. Great work! Thank you for contributing to HackForLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update HTML components to MUI: ./src/components/admin/reports/index.js
2 participants