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

Refactor CSS in src/components/AddPeopleToTag/AddPeopleToTag.tsx #3155

Closed
rahulch07 opened this issue Jan 5, 2025 · 5 comments
Closed

Refactor CSS in src/components/AddPeopleToTag/AddPeopleToTag.tsx #3155

rahulch07 opened this issue Jan 5, 2025 · 5 comments
Labels
bug Something isn't working good first issue Good for newcomers refactor Refactor the code to improve its quality ui/ux issue related and being worked with the figma file of the Admin UI unapproved

Comments

@rahulch07
Copy link
Contributor

Refactor CSS in this file

To improve the UI/UX of Talawa-Admin we have been working on:

  1. making the application more suitable to color blind users
  2. streamlining all CSS into a single global file

This file contains embedded CSS references that should be completely moved to:

  • src/style/app.module.css

Additional Information:

  1. The foundational work for this was completed in this PR:
    -Chore/css changes #2466
  2. The desired single CSS file is:
    • src/style/app.module.css

The Challenge:

The goal is for this file to use the new design pattern. This means that:

  1. All embedded CSS in this file must be merged into the global CSS file using pre-existing CSS.
  2. All embedded CSS in this file must be removed.

Related PR

#2984

@rahulch07 rahulch07 added the bug Something isn't working label Jan 5, 2025
@github-actions github-actions bot added refactor Refactor the code to improve its quality ui/ux issue related and being worked with the figma file of the Admin UI unapproved good first issue Good for newcomers labels Jan 5, 2025
@rahulch07
Copy link
Contributor Author

please assign

@palisadoes
Copy link
Contributor

Unassigning so you can work on the follow up for this:

@palisadoes
Copy link
Contributor

We created an this issue to do basic changes to the CSS colors for consistency

The resulting PR created a comprehensive design system for the entire app. This was unexpected. The approach was a long term goal that became immediately implementable. We had to act promptly.

We have just merged the PR. There will be a follow up PR to adjust some colors as the Figma design has lettering that is sometimes hard to read.

When this is done we'll need your assistance to merge the screens into the develop branch with this issue.

If your screens have not been affected, then you should use the new methodology to refactor them.

This is an extraordinary circumstance. I hope you will appreciate the decision.

1 similar comment
@palisadoes
Copy link
Contributor

We created an this issue to do basic changes to the CSS colors for consistency

The resulting PR created a comprehensive design system for the entire app. This was unexpected. The approach was a long term goal that became immediately implementable. We had to act promptly.

We have just merged the PR. There will be a follow up PR to adjust some colors as the Figma design has lettering that is sometimes hard to read.

When this is done we'll need your assistance to merge the screens into the develop branch with this issue.

If your screens have not been affected, then you should use the new methodology to refactor them.

This is an extraordinary circumstance. I hope you will appreciate the decision.

@rahulch07
Copy link
Contributor Author

@palisadoes this issue is completed. As issue #2880 required these files to be changed. Therefore the solution for this was submitted in PR #2984 . Solution can be seen here https://github.com/PalisadoesFoundation/talawa-admin/pull/2984/files#diff-5dfc709eb0fbff3a1aa2dca32ca5eca3cbb3e19dc84c3c8526d9e854460042f5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers refactor Refactor the code to improve its quality ui/ux issue related and being worked with the figma file of the Admin UI unapproved
Projects
Status: Done
Status: Done
Development

No branches or pull requests

2 participants