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

Improvement for k8s.io/docs/concepts/security/security-checklist/ #49588

Open
rafael-f opened this issue Jan 29, 2025 · 6 comments
Open

Improvement for k8s.io/docs/concepts/security/security-checklist/ #49588

rafael-f opened this issue Jan 29, 2025 · 6 comments
Labels
needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.

Comments

@rafael-f
Copy link

The disabled property of the checkbox inputs on this page could be removed to make them interactible.

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Jan 29, 2025
@k8s-ci-robot
Copy link
Contributor

This issue is currently awaiting triage.

SIG Docs takes a lead on issue triage for this website, but any Kubernetes member can accept issues by applying the triage/accepted label.

The triage/accepted label can be added by org members by writing /triage accepted in a comment.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@utkarsh-singh1
Copy link
Contributor

I agree to the decision to make it more intractable, but I want to make let users understand more about this page and what it provides.

@utkarsh-singh1
Copy link
Contributor

As the statement at the top of the docs says -

This checklist aims at providing a basic list of guidance with links to more comprehensive documentation on each topic. It does not claim to be exhaustive and is meant to evolve.

But checklist does not seem to work and some of the explanation to the checklist topics are present already with their links to the relative document.

@sftim
Copy link
Contributor

sftim commented Jan 29, 2025

/priority awaiting-more-evidence

Why do we want people to check these items? The original intend was that they wouldn't be able to.

@k8s-ci-robot k8s-ci-robot added the priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. label Jan 29, 2025
@rafael-f
Copy link
Author

If you can check the items you can more easily navigate away to study each of the topics then come back, check it, and move to the next one. Similarly to what online courses usually do.

When you see a checklist you expect to be able to check the items, otherwise it's just a bullet list.

If it's feared that users might confuse the list from a suggested list to a extensive list then renaming the page and removing the checkboxes could be a better way.

@utkarsh-singh1
Copy link
Contributor

Why do we want people to check these items? The original intend was that they wouldn't be able to.

Well, means we can remove the checkbox as @rafael-f also stated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done.
Projects
None yet
Development

No branches or pull requests

4 participants