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

Use the new logo in Kubeapps dashboard #5130

Merged
merged 7 commits into from
Jul 27, 2022

Conversation

antgamdia
Copy link
Contributor

Description of the change

This PR updates the logo in several places on the Kubeapps dashboard, namely: favicons, internal icons, header logo.

Benefits

The new logo will be consistently used.

Possible drawbacks

The dark mode trick (inverse colors) doesn't really work here. We will probably have to revisit it.

Applicable issues

Additional information

image

image

image

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@netlify
Copy link

netlify bot commented Jul 26, 2022

Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 486aea9
🔍 Latest deploy log https://app.netlify.com/sites/kubeapps-dev/deploys/62e0049c27e94b000a59b667

@antgamdia antgamdia mentioned this pull request Jul 26, 2022
8 tasks
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>

Conflicts:
	dashboard/src/components/NotFound/__snapshots__/NotFound.test.tsx.snap
Copy link
Collaborator

@castelblanque castelblanque left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia merged commit 1478cd5 into vmware-tanzu:main Jul 27, 2022
@antgamdia antgamdia deleted the 5120-new-logo-app branch July 27, 2022 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants