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

chore: extract color palette to its own file #5931

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

benoitf
Copy link
Collaborator

@benoitf benoitf commented Feb 9, 2024

What does this PR do?

make the palette color being more embeddable elsewhere or reusable
it is like a no-op change

sort colors alphabetically

Screenshot / video of UI

What issues does this PR fix or reference?

related to:

How to test this PR?

Podman Desktop or website colors should be the same as before

make the palette color being more embeddable elsewhere
or reusable

it is like a no-op change

related to containers#5878
containers#4907

Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf requested a review from a team as a code owner February 9, 2024 17:14
@benoitf benoitf requested review from jeffmaury and feloy and removed request for a team February 9, 2024 17:14
Copy link
Collaborator

@deboer-tim deboer-tim left a comment

Choose a reason for hiding this comment

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

I'm curious where you're going with this, but since it is a safe and simple change I'm happy to approve.

@benoitf benoitf merged commit df124be into containers:main Feb 10, 2024
8 checks passed
@benoitf benoitf deleted the extract-color-palette branch February 10, 2024 18:09
@podman-desktop-bot podman-desktop-bot added this to the 1.8.0 milestone Feb 10, 2024
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.

None yet

4 participants