Skip to content

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

Merged
thesocialdev merged 2 commits intostagefrom
Migrate-components-used-in-the-create-report-page-to-mui
Jan 18, 2025
Merged

Migrate components used in the create report page to mui#1783
thesocialdev merged 2 commits intostagefrom
Migrate-components-used-in-the-create-report-page-to-mui

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
@sonarqubecloud
Copy link

@thesocialdev thesocialdev merged commit 93182f5 into stage Jan 18, 2025
6 of 7 checks passed
@thesocialdev thesocialdev mentioned this pull request Jan 26, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment