-
-
Notifications
You must be signed in to change notification settings - Fork 893
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
NoMongo: Fix Stylesheet in Anticipation of Dark Mode #3296
Comments
Hy @palisadoes, can i work on this issue ? |
Also remove any references that are unused |
I will collaborate with @hustlernik to devise a strategy for working more efficiently. Once finalized, we’ll share it so others can follow as well. |
This issue did not get any activity in the past 10 days and will be closed in 180 days if no update occurs. Please check if the develop branch has fixed it and report again or close the issue. |
@hustlernik @rahulch07 Have you been able to work on this? I just had to close a duplicate issue related to this |
@palisadoes Steps:
Likewise this can be followed with all the components and others can also do this on the components they are working. would this work? |
Yes. Thanks |
@palisadoes I am working to refactor the CSS for screens whose mongo to postgres migration is completed. |
Thanks.
|
A simple "search and replace" is not enough. Please read carefully
Is your feature request related to a problem? Please describe.
--red-delete-bg
use--delete-button-bg
--red-delete-text
use--delete-button-text
--blue-primary
use a name that describes the role of the color like--margin-color-for-xxxxxxxx
wherexxxxxxx
is the component--brown-color
occur 25 times in different roles in different screens and components.--light-dark-green
occur 3 times in similar situations. The role in different screens and components should be used instead.--black-color
occur 36 times in similar situations. The role in different screens and components should be used instead.app-dark.module.css
file for Dark Mode the process will be much easier.Additional context
Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship
The text was updated successfully, but these errors were encountered: