kubernetes / website

Kubernetes website and documentation repo:
https://kubernetes.io
Creative Commons Attribution 4.0 International
4.46k stars 14.36k forks source link

Expand style guide #7030

Open steveperry-53 opened 6 years ago

steveperry-53 commented 6 years ago

This is a...

Problem: The current documentation style guide is missing some key information.

The current style guide has some guidelines that I would like to revisit.

Proposed Solution:

Add more guidance in these areas:

Revisit these areas:

Bring the style guide into alignment with itself.

Example: The copy is called a "fork" should be The copy is called a fork.

Page to Update: https://kubernetes.io/docs/home/contribute/style-guide/

heckj commented 6 years ago

:+1: sounds good to me

jesscodez commented 6 years ago

These guidelines make sense overall. Though I was guilty of breaking quite a few of them in the app developer user journey, I revised most violations before thet PR merged. (A few "air quotes" did slip through.)

That being said, I think that the rule around bolded text warrants more discussion. Doc writers should be able to use bold for visual effect, not just for GUI elements. When readers are scrolling quickly through a page, bold helps them easily pick out the terms or phrases that are important.

There's potential for over-bolding, but we can use the style guide to caution people to use their best judgment.

bradtopol commented 6 years ago

I think the proposed improvements and desire to revisit certain areas would be very helpful

Bradamant3 commented 6 years ago

I see @abiogenesis-now's point about wanting emphasis. But it's all too common to wind up with too much bold on the page. And it loses its effectiveness in displaying names of GUI elements. Do we have any other options for emphasis?

jesscodez commented 6 years ago

I'm not sure what styling would be best, but I'm also okay with creating a special CSS class---having to write <span class="special-styling">Text</span> creates enough of a barrier for entry that it won't be overused (?)

Bradamant3 commented 6 years ago

I'd also like to see us think not just about revising the guidelines, but also about promoting them more effectively. We could provide some small-scale enforcement with a doc linter like vale, but maybe we also need to think about ways to explain more specifically what we mean by some of the writing recommendations we make (as opposed to the markup recommendations) -- that is, the items in the current style guide listed under Content best practices and Patterns to avoid (https://kubernetes.io/docs/home/contribute/style-guide/#content-best-practices).

We could also brainstorm ways to take the style guide back to the rest of the community. Part of the sig-docs-to-other-sigs initiative, and the docs training as part of Paris's mentoring program -- although we should probably get better alignment in practice within sig-docs itself before we take things out to a larger group.

That said, we do need to consider what makes sense to try to enforce.

steveperry-53 commented 6 years ago

Thanks everybody. Keep those comments coming. I've been looking over the new user journeys content. I see fairly consistent use of things that go against our style guide. So I think it's important for us to discuss this and come to agreement.

Here's just one example. The phrase "we'll show ...", goes against two of our current guidelines: Use present tense, and don't use "we".

steveperry-53 commented 6 years ago

To answer the question about options for emphasis: The style guide has a section on Callout Formatting.

heckj commented 6 years ago

Where was the “we’ll show” phrase? I was expecting I was probably the one that violated that style guideline, as I have a tendency to use it informally to build a sense of companionship with the reader. I totally get WHY we don’t want to use it, so I tried to not use it - that said, I didn’t spot it in a quick look. We may not have caught it on the updates/edits.

steveperry-53 commented 6 years ago

Here's a "we'll get". https://kubernetes.io/docs/user-journeys/users/cluster-operator/intermediate/

Seems like I've spotted other cases of "we'll". Not sure it's worth hunting them all down right now.

Here's a "will introduce". https://kubernetes.io/docs/user-journeys/users/cluster-operator/foundational/

heckj commented 6 years ago

Heh - it was me, I suspected it might be. I’ll take a pass re-reading that whole set and make the relevant updates to match our style guide. Thank you

steveperry-53 commented 6 years ago

/lifecycle frozen

sftim commented 5 years ago

/priority backlog Is that OK?

zacharysarah commented 5 years ago

@sftim I think the backlog is a good place for this--and @Bradamant3's recommendation of vale may be remarkably prescient.

https://errata-ai.github.io/vale/ https://github.com/apps/vale-linter

sftim commented 5 years ago

Sounds good to me!

sftim commented 3 years ago

https://github.com/kubernetes/website/issues/20862 and https://github.com/kubernetes/website/issues/23527 are relevant here.

sftim commented 11 months ago

/triage accepted

kumarankit999 commented 11 months ago

Should I work on this @sftim

sftim commented 10 months ago

Should I work on this

Anyone is welcome to work on this. It's useful work that helps other contributors be more effective, so help is appreciated!

kumarankit999 commented 10 months ago

/assign

sftim commented 3 months ago

/remove-lifecycle frozen

If it rots, it rots.