Currently, mobile is using an outdated FontistoIcon component, which needs to be replaced with the new Icon component.
This is a massive undertaking by itself and assigning to a single contributor and creating a single PR would be too large. Smaller PRs can be submitted against this issue from multiple contributors to ensure easier review and gradual improvements.
Technical Details
Replace instances of FontistoIcon component with Icon component (app/component-library/components/Icons/Icon/Icon.tsx)
Component APIs are slightly different so ensure all props have been migrated appropriately
Acceptance Criteria
Instances of FontistoIcon component are completely replaced with the new Icon component
The component APIs are updated to reflect the changes in the new Icon component and there is no functional changes or visual regression
Each Pull Request (PR) should include no more than 3 files
PR descriptions must adhere to the designated template, ensuring that all relevant sections are completed including before and after screenshots of UI.
The code changes should pass Jest tests, e2e tests, linting, and Storybook without any errors.
If the acceptance criteria is not met, PRs may be closed.
Difficulty: Intermediate
Good first issue for: External contributors who are familiar with running mobile locally, have knowledge of React Native, component props, Jest tests, linting, and Storybook, and want to contribute to improving the cohesiveness of UI in mobile
Description
Currently, mobile is using an outdated
FontistoIcon
component, which needs to be replaced with the newIcon
component.This is a massive undertaking by itself and assigning to a single contributor and creating a single PR would be too large. Smaller PRs can be submitted against this issue from multiple contributors to ensure easier review and gradual improvements.
Technical Details
FontistoIcon
component withIcon
component (app/component-library/components/Icons/Icon/Icon.tsx
)Acceptance Criteria
FontistoIcon
component are completely replaced with the newIcon
componentIcon
component and there is no functional changes or visual regressionIf the acceptance criteria is not met, PRs may be closed.
Difficulty: Intermediate
Good first issue for: External contributors who are familiar with running mobile locally, have knowledge of React Native, component props, Jest tests, linting, and Storybook, and want to contribute to improving the cohesiveness of UI in mobile