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

Support logo in starlark markup #1872

Closed
Dartoxian opened this issue Nov 30, 2023 · 4 comments
Closed

Support logo in starlark markup #1872

Dartoxian opened this issue Nov 30, 2023 · 4 comments
Labels
feature request frontend Label for issues (bugs, FRs, etc) that pertain to the Kurtosis frontend nice to have Nice to have feature

Comments

@Dartoxian
Copy link
Contributor

Background & motivation

Our designs allow for a custom package logo in the catalog cards
image

Desired behaviour

Please could support for a logo (as a public url?) be added to the package definition and returned in the package indexer.

How important is this to you?

Nice to have; this feature would make using Kurtosis more enjoyable.

What area of the product does this pertain to?

Frontend: the Graphical User Interface, including the Enclave Manager

@github-actions github-actions bot added frontend Label for issues (bugs, FRs, etc) that pertain to the Kurtosis frontend nice to have Nice to have feature labels Nov 30, 2023
@mud1tx
Copy link

mud1tx commented Nov 30, 2023

@Dartoxian I can work on this but can you explain a little more.

@Dartoxian
Copy link
Contributor Author

Hi @mud1tx - the screenshot above is a mockup of how the enclave manager ui catalog view should look. In it each package is able to have its own custom logo, and if it's not present then we just use a kurtosis logo.

The logo presumably should be defined in the starlark markup so that https://github.com/kurtosis-tech/kurtosis-package-indexer is able to return the logo as a full url in its KurtosisPackage object (presumably kurtosis-package-indexer needs to be updated as well).

@leoporoli
Copy link
Contributor

@mud1tx This is a backend task on this repository https://github.com/kurtosis-tech/kurtosis-package-indexer
it's a Golang GRPC server that contains two endpoints returning the kurtosis package basically what you have to do is add a new field on this type which will return the "Kurtosis Package Icon" image URL, so the FE can get this URL and download the image.

The BE should check if the image exist in the expected repo path (it should be next to the kurtosis.yml file, in the same folder and the name should be kurtosis-package-icon.png) if the image doesn't exist it could return the GitHub org image instead and if it's empty, it should return an empty string.

@mud1tx let me know if you need more details

@leoporoli
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request frontend Label for issues (bugs, FRs, etc) that pertain to the Kurtosis frontend nice to have Nice to have feature
Projects
None yet
Development

No branches or pull requests

3 participants