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

replaced html with mui in index #1748

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

Conversation

nora-zajzon
Copy link
Member

Fixes #1679

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

  • Changed all the HTML components to MUI applicable components
  • Used css files to format instead of inline sx to style like @jbubar requested. Caused some visual changes in font and what not. Let me know if this is an issue

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied Screenshot 2024-09-17 at 11 04 02 AM
Visuals after changes are applied Screenshot 2024-09-17 at 10 57 45 AM

@nora-zajzon
Copy link
Member Author

@JackHaeg I made the requested changes for this pr here not sure if i can close the other or if it still has pending merges

@JackHaeg
Copy link
Member

@nora-zajzon Thank you for making those changes to this PR! Can you please comment directly on the other PR you are referencing above, tag Trillium, and ask him if you can close it?

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