[UI] Refactored to actually use colors from Nativewind - Githubissues

Educado-App / educado-mobile

The repository for the educado mobile education application

GNU General Public License v3.0

1 stars 3 forks source link

[UI] Refactored to actually use colors from Nativewind #209

Closed ITPini closed 11 months ago

ITPini commented 11 months ago

Description

This update aims to fix a lot of wrong use of color variables from Tailwind config.

Changes

Fixed MANY components using wrong Tailwind config variables
Refactored various components using unused styling
Updated relevant unit tests
Dependency added: none

Related Issues

Nativewind styling does not seem to be applied #161

Checklist

[x] Code has been tested locally and passes all relevant tests.
[x] Documentation has been updated to reflect the changes, if applicable.
[x] Code follows the established coding style and guidelines of the project.
[x] All new and existing tests related to the changes have passed.
[x] Any necessary dependencies or new packages have been properly documented.
[x] Pull request title and description are clear and descriptive.
[x] Reviewers have been assigned to the pull request.
[x] Any potential security implications have been considered and addressed.
[x] Performance impact of the changes has been evaluated, if relevant.

If mobile/frontend pull request, what version of the backend is it stable, and running on?

Branch: dev

Notes for Reviewers

I deliberately didn't change the styling on deprecated components. I might have missed some places where it still uses wrong variables