-
Notifications
You must be signed in to change notification settings - Fork 2
feat add infrastructure ref kind to clusters #1302
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
Conversation
Signed-off-by: Rana Tarek Hassan <[email protected]>
Signed-off-by: Rana Tarek Hassan <[email protected]>
…usters Signed-off-by: Rana Tarek Hassan <[email protected]>
Signed-off-by: Rana Tarek Hassan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Looks like there might be an unintentional whitespace change in the TS imports, but otherwise it looks great!
Signed-off-by: Rana Tarek Hassan <[email protected]>
WRT Icons!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is wicked very nicely done 🙌 .
We also use prettier
to do all the autoformatting, (like gofmt for go). So if using vscode you can install the prettier extension and turn on format-on-save.
Updates in clusterDashboard Co-authored-by: Simon <[email protected]>
Signed-off-by: Rana Tarek Hassan <[email protected]>
Signed-off-by: Rana Tarek Hassan <[email protected]>
@bia the changes for using the existing icons in the clusters list are added here |
cc @bia do we have an alternative icon for k8s? https://fontawesome.com/icons/steering-wheel?s=solid ? |
Let's use an unaltered version of the k8s logo. |
On the
Notes:
|
In the case of
if the capi cluster infrstructure ref kind is |
Yes! capi/other is not that useful. So we could change the type column to show the infraRef/explict-annotation value instead. The column could be just an icon with a `title="type: DockerCluster" so we can see more on hover..
|
Let’s do it! I’m sketching this. |
I'm still waiting on an answer from [email protected] to confirm that we can use a grey k8s logo, but here is my proposal, does it make sense? I'm using the fa6 wine bottle, which I think matches material a bit better. Is there a licencing issue with that? |
Currently the kind icon is updated with the new bottle, and the icons are moved to the type column with the grey color for all. (Including the k8s logo until we receive an answer regarding the license) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wicked! ⭐ , looking great.
Co-authored-by: Simon <[email protected]>
Adjust kind cluster icon size Signed-off-by: Rana Tarek Hassan <[email protected]>
Signed-off-by: Rana Tarek Hassan <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beaut! ✨ , this looks great!
Issue
Fixes #1165
Description
Example: