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

Change all antd icons #1680

Merged
merged 1 commit into from
Jan 7, 2025
Merged

Change all antd icons #1680

merged 1 commit into from
Jan 7, 2025

Conversation

LuizFNJ
Copy link
Collaborator

@LuizFNJ LuizFNJ commented Dec 21, 2024

Description

In this PR, I replaced all the icons used from Ant Design with MUI icons. As I worked on the components, I also replaced some Col, Row, and Typography elements with MUI's Grid and Typography.

Testing

I only replaced the icons, but in some components, I also adjusted the responsiveness since I was already working on them.

Icons and responsive :

Just icons:

A visual document outlining which visual parts of the site are intended to facilitate testing:
testeicons.pdf

Closes #1632, #1674, #1681, #1647, #1644, #1626, #1574, #1510, #1651, #1663, #1633, #1642, #1669, #1641, #1668, #1671, #1645, #1636, #1665, #1673, #1648, #1672, #1646, #1631, #1630, #1629, #1628, #1627, #1625, #1624, #1623, #1621, #1620, #1618, #1617, #1616, #1615, #1614, #1682, #1654, #1653, #1652, #1650, #1649, #1640, #1637, #1635, #1634

@LuizFNJ LuizFNJ force-pushed the Change-all-icons-antd branch from 85249c3 to aafbdd9 Compare December 21, 2024 20:03
@thesocialdev thesocialdev merged commit 059b0ea into stage Jan 7, 2025
7 checks passed
@thesocialdev thesocialdev deleted the Change-all-icons-antd branch January 7, 2025 15:50
@thesocialdev
Copy link
Collaborator

@LuizFNJ you shouldn't use comma when using "Closes #NUMBER_OF_ISSUE1 #NUMBER_OF_ISSUE2 #NUMBER_OF_ISSUE" you'll need to close them manually

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

Successfully merging this pull request may close these issues.

Migrate component BaseList.tsx - Import { LoadingOutlined, Button, Col, Row }
2 participants