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(Organizational Chart): fixed the UI error in dark theme #2829

Open
wants to merge 1 commit into
base: version-15-hotfix
Choose a base branch
from

Conversation

thomasantony12
Copy link
Contributor

@thomasantony12 thomasantony12 commented Mar 2, 2025

  • fixed the UI error in dark theme

Before

Screenshot (204)

After

Screenshot (205)

close #1142

@thomasantony12
Copy link
Contributor Author

@ruchamahabal can you look into this.

@asmitahase
Copy link
Collaborator

asmitahase commented Mar 7, 2025

Hi @thomasantony12, could you please add before and after screenshot since this is a UI fix?
It makes reviewing much faster for us. Thanks!

@thomasantony12
Copy link
Contributor Author

@asmitahase i will add it shortly

@thomasantony12
Copy link
Contributor Author

@asmitahase The screen shots are attached to the commit description. please look into it and tell if there is anything else.

@thomasantony12
Copy link
Contributor Author

@asmitahase is have seen the changes?

@thomasantony12
Copy link
Contributor Author

@asmitahase @ruchamahabal any update on the PR.

@ruchamahabal
Copy link
Member

ruchamahabal commented Mar 12, 2025

@thomasantony12 linter is failing due to extra trailing spaces in the file. Can you fix it?

image

@thomasantony12 thomasantony12 force-pushed the orgnizational_chart_theme_fix branch from cb94943 to bf947db Compare March 12, 2025 15:33
@thomasantony12
Copy link
Contributor Author

@ruchamahabal changes are made

@thomasantony12 thomasantony12 force-pushed the orgnizational_chart_theme_fix branch from bf947db to 645ccd3 Compare March 12, 2025 15:42
@thomasantony12 thomasantony12 force-pushed the orgnizational_chart_theme_fix branch from 645ccd3 to 2a3282d Compare March 16, 2025 07:56
@thomasantony12
Copy link
Contributor Author

@ruchamahabal

1 similar comment
@thomasantony12
Copy link
Contributor Author

@ruchamahabal

@thomasantony12
Copy link
Contributor Author

@asmitahase

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.

3 participants