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

KU-2068 reformatted annotations table #789

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

nhennigan
Copy link
Contributor

Due to formatting issues with large amounts of text, the annotations table was quite unclear. Edited to make it more readable

due to formatting issues the annotations table was quite unclear. Edited to make it more readable
@evilnick
Copy link
Contributor

This is a great improvement! I still think we can do better though

adding example of how to include annotations in your cluster-config yaml file as requested in PR review
@evilnick
Copy link
Contributor

Update - there is no simple way to create the headings we want.

The 'proper' way to do it would be to define a new style for the h2 headings and then apply it only to the headings on this page. However, the 'proper' way to do this is using the sphinx directive - cssclass.
Sadly, that works on pretty much everything except headers

My next experiment was to create a div so I could apply a class that way and use that to style the headers. Sadly, even the ugly way of adding HTML to do this seems to break the theme (the footer floats into space).

I think that leaves us with - the hacky way and the ugly, hacky way

The hacky way is to include a blob of JS on this page which will rewrite the styles for headings, which I will try next.

@evilnick
Copy link
Contributor

i think we can maybe bump up the size to 1.5em

Co-authored-by: Nick Veitch <nick.veitch@canonical.com>
k8sd/v1alpha/lifecycle/skip-cleanup-kubernetes-node-on-remove: true
k8sd/v1alpha/lifecycle/skip-stop-services-on-remove: true
```

Copy link
Contributor

Choose a reason for hiding this comment

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

might be nice to have some text here to ease the transition to the actual list of annotations

include link to kubernetes website with more info on annotations, make []string clearer to non golang users and include line between eg and list
@nhennigan nhennigan marked this pull request as ready for review November 15, 2024 17:16
@nhennigan nhennigan requested a review from a team as a code owner November 15, 2024 17:16
@bschimke95
Copy link
Contributor

Please rebase this PR.

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.

4 participants