-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Remove redundant, out of sync table of contents #3433
Conversation
This table of contents is redundant (it's also available from the side nav) and confusing (not clear if it's a copy of the side nav or contains additional information). Plus, it was out of sync with the actual table of contents (last section on GPU was missing). So, I think it's best to just remove it.
Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA. It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.
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/test-infra repository. I understand the commands that are listed here. |
@olalonde IMHO, it would be best if it were updated, not just removed. Otherwise, there's not much content on the Tasks landing page, which might be confusing. |
@steveperry-53 Do you want to chime in? |
I can't talk for everyone but I personally feel keeping the page empty (or with a clear "Next step" link) is a better UX. Here's what I did:
Ok.. it wasn't that dramatic but you get the idea 👍 |
Empty page is ok, but speaking of UX side:
|
Ouch. Missed the critical part - the empty page doesn't explain what "Kubernetes Tasks" really are. |
It looks like the "Tasks" section of documentation is a "Cookbook" with recipes. Not really something embedded in Kubernetes. |
I'll work on rewriting the copy for the explanation of what Tasks are and maybe a high-level description of each grouping in the ToC. |
@chenopis, I'm fine with whatever solution you and @techtonik work out. |
Address #3433 - update Tasks landing page
@olalonde @techtonik I merged this PR and added some copy in #3593. Let me know if that works for you. |
@chenopis yes, that's better I think. |
…hub.io into chenopis-weaveworks-tabs * 'master' of https://github.com/kubernetes/kubernetes.github.io: Fixes broken links Removed deleted links regarding minikube CLI Add missing volumetype to pv doc (#3600) Fix broken link in HA doc with minikube (#3599) rolling-update: remove link to the doc itself Link to 'why containers' in kubernetes-basics Updating Ubuntu based instructions and upgrades Address Issue #3577 Added more explanation for nodeAffinity. Update and fix travis tests (short-term) Address #3433 - update Tasks landing page Remove redundant, out of sync table of contents
…hub.io into chenopis-weaveworks-tabs * 'master' of https://github.com/kubernetes/kubernetes.github.io: (258 commits) Fixes broken links Removed deleted links regarding minikube CLI Add missing volumetype to pv doc (#3600) Fix broken link in HA doc with minikube (#3599) rolling-update: remove link to the doc itself Link to 'why containers' in kubernetes-basics Updating Ubuntu based instructions and upgrades Address Issue #3577 Added more explanation for nodeAffinity. Update and fix travis tests (short-term) Address #3433 - update Tasks landing page Remove redundant, out of sync table of contents Update Init Container link Update service.md Link to appropriate docs for Automatic Mounting of Manually Created Secrets (#3571) update kubectl apply doc for required resource name jlx426 Updating Canal URL (#3583) Grammar patch to #3572 Remove swagger-ui from search index ...
This table of contents is redundant (it's also available from the side nav) and confusing (not clear if it's a copy of the side nav or contains additional information). Plus, it was out of sync with the actual table of contents (last section on GPU was missing). So, I think it's best to just remove it.
This change is