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

Dark mode issue for invite user with email #7513 #7532

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Yash-rai-29
Copy link

Pull Request Description

Fixes #7513

Issue Resolved
This pull request addresses the dark mode issue for the "Invite user with email" feature, as reported in issue #7513.

Proposed Changes
To enhance visibility in dark mode, the icon color for the "Email" feature has been updated to a lighter shade, specifically white, ensuring improved contrast and visibility.

Steps to Verify
Navigate to the workspace settings.
Select "Users" and click "Add Users."
Proceed to invite users via email and observe the icon color in dark mode.
Screenshots
Attached screenshots showcasing the updated icon color in dark mode.
Before resolving the issue: -
image

After resolving the issue: -
image

Version Information
Previous Version: 2.17.5-ee2.5.10-cloud2.0.9
Updated Version: [Insert version with the proposed changes]
Additional Notes
This change is aimed at improving the user experience in dark mode by addressing the visibility concerns for the "Invite user with email" feature.
Any feedback or further adjustments are welcomed for an optimal resolution of the reported issue.

@CLAassistant
Copy link

CLAassistant commented Sep 28, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@kavinvenkatachalam kavinvenkatachalam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HEy @Yash-rai-29 ,

Thanks for the PR. It's working fine in darkmode but not able to see the icon in light mode. Could you please look into it? Attaching the screenshot

image

@Yash-rai-29
Copy link
Author

@kavinvenkatachalam
Thanks for the heads up. I'll definitely look into the icon visibility in light mode and make the necessary adjustments to ensure it works seamlessly across both dark and light modes.

Appreciate your feedback and the screenshot!

@Yash-rai-29
Copy link
Author

👋 Hi @kavinvenkatachalam,

Just a friendly reminder regarding the pull request on the fix/dark-mode-icon branch in the Yash-rai-29/ToolJet repository. @Yash-rai-29 requested your review 4 days ago. There's one change requested, and we'd greatly appreciate your insights and feedback.

Copy link
Collaborator

@kavinvenkatachalam kavinvenkatachalam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yash-rai-29 - On clicking the Add Users nothing happens. The recent change breaking the functionality. Please take a look

Copy link
Collaborator

@kavinvenkatachalam kavinvenkatachalam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Yash-rai-29 On clicking Add users button nothing happens. Please take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dark mode issue for invite user with email
3 participants