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(images): images suited for black background #306

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shrutimantri
Copy link
Contributor

No description provided.

Copy link
Member

@wrussell1999 wrussell1999 left a comment

Choose a reason for hiding this comment

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

@shrutimantri is there a way to get the IBM DB2 logo to not have the hexagon?

Copy link
Member

@wrussell1999 wrussell1999 left a comment

Choose a reason for hiding this comment

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

You need to update the icon to handle black / white theme, example of wrong [one](https://raw.githubusercontent.com/kestra-io/kestra/6de50afeca9600dbf1c528f7daa5a3bf9a664e82/core/src/main/resources/icons/io.kestra.core.tasks.flows.WaitFor.svg), good [one](https://raw.githubusercontent.com/kestra-io/kestra/d28b63de0feef1ea629ce04e76987af4f6076c1d/core/src/main/resources/icons/io.kestra.core.tasks.flows.Pause.svg). The main difference is :
fill="currentColor"  
that allow to change the color of the svg.
Please take care of this?

@tchiotludo tchiotludo force-pushed the master branch 2 times, most recently from 35f94a7 to 7e25554 Compare June 12, 2024 13:18
@brian-mulier-p brian-mulier-p force-pushed the master branch 7 times, most recently from 879b3d5 to aa6ca11 Compare August 13, 2024 12:18
@tchiotludo tchiotludo force-pushed the master branch 2 times, most recently from cae2ae8 to 7fee41c Compare September 24, 2024 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To review
Development

Successfully merging this pull request may close these issues.

2 participants