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

fix: Changed footer text color to darker shade on hover in light mode #196

Closed
wants to merge 0 commits into from

Conversation

sharmashekhar0
Copy link

@sharmashekhar0 sharmashekhar0 commented Oct 19, 2023

What does this PR do?

This PR simply changes the hover effect of footer items in the light mode. Previously text color changed to "text-gray-100" on hover in light mode which made text almost invisible. This PR fixed this issue and now text color changes to darker shade on hover in light mode and text is clearly visible.

Fixes #195

[Library | Loom - 19 October 2023 - Watch Video](https://www.loom.com/share/882a4d9bdf1e45c39f7d4bba8fc84d89)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

This PR changes just the hover effect on footer items. These changes can be visually test by going to the footer of review app and hovering over footer items.

@github-actions
Copy link

github-actions bot commented Oct 19, 2023

Thank you for following the naming conventions! 🙏

@sharmashekhar0 sharmashekhar0 changed the title Fix: Changed footer text color to darker shade on hover in light mode fix: Changed footer text color to darker shade on hover in light mode Oct 19, 2023
@sharmashekhar0
Copy link
Author

I think i needed to include some files in gitignore but I am not sure what needs to be in gitingore file. So if there is some change needed just comment that down.

@github-actions
Copy link

Hey there, there is a merge conflict, can you take a look?

@github-actions github-actions bot added the 🚨 merge conflict Merge Conflict label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚨 merge conflict Merge Conflict
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Footer text color changes to lighter color in light mode when hovering
1 participant