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

Migrate components used in the create report page to mui #1783

Merged

Conversation

LuizFNJ
Copy link
Collaborator

@LuizFNJ LuizFNJ commented Jan 18, 2025

Description

In this PR, I migrated the components used in the create report page to MUI Material. This mostly involved replacing Col and Row with Grid and adapting the Typography from Ant Design to MUI. Also, the MUI tag was replaced with the Chip, and the buttons from Ant Design were replaced with IconButton because they are buttons that house icons.

Testing

In this PR, I managed to separate the pages as we discussed. The Col and Row were replaced by Grid, and in relation to this, the site's responsiveness needs to be tested. You can test the Chip by adding topics, while the IconButton tests the functionality and design of the buttons. All the Typography from Ant Design was also replaced, so it’s necessary to check if there is any text that’s out of the pattern or breaking the design.

Components:

A visual document outlining which visual parts of the site need to be tested:
Guia visual testes 1.pdf
This guide was created while I was making the changes. You’ll be able to use it in the following PRs as well.

closes #1730 , closes #1747 , closes #1729 , closes #1751 , closes #1743 , closes #1749 , closes #1742 , closes #1503

@LuizFNJ LuizFNJ requested a review from thesocialdev as a code owner January 18, 2025 13:39
@LuizFNJ LuizFNJ force-pushed the Migrate-components-used-in-the-create-report-page-to-mui branch from c092f6e to 5de5cac Compare January 18, 2025 14:08
@thesocialdev thesocialdev merged commit 93182f5 into stage Jan 18, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment