Skip to content
This repository has been archived by the owner on Oct 8, 2020. It is now read-only.

Docs style guide changes and improvements #503

Merged
merged 3 commits into from
Apr 3, 2018

Conversation

seancolsen
Copy link
Contributor

Three distinct changes to the style guide, as described in each commit.

I'd like some feedback from these folks before continuing:

This makes it easier to refer people to specific guidelines within the
guide.
Some places in the Dev and Sysadmin guides have deviated from this
guideline and I think it's for the better.

Examples:

* https://docs.civicrm.org/dev/en/latest/framework/angular/
* https://docs.civicrm.org/sysadmin/en/latest/integration/drupal/views/
Use the terminology "Chapter/Sub-chapter/Page/Section" instead of
"Part/??/Chapter/Section" because it's more intuitive.
@totten totten changed the title Doces style guide changes and improvements Docs style guide changes and improvements Mar 27, 2018
@joannechester
Copy link

I agree that 4 levels of menu can be needed for some topics. I was thinking I needed that extra level when planning the reorganisation of the email chapter. It makes no sense to have A/B testing at the second menu level as it is only relevant to CiviMail. so we need Email - Mass mailings using CiviMail - A/B testing

I am not so keen on the nomenclature for the 4 levels. To me "page" doesn't fit. I think of a page as the place the information is displayed, not as a part of the content that is being displayed. This is obviously true for a physical book, but I think it also applies online where a page can have a heading, side bars etc. The content is just one part of it.

However, having spent a couple of hours trawling on this topic, I can't find a definitive alternative. As it fits will how things are already described in the documentation guide so I am OK with it.

@dvhirst
Copy link
Contributor

dvhirst commented Apr 3, 2018

Per discussion, this seems very sensible to me. Glad to have a definitive 1 - 1 relationship specified. Don

@seancolsen
Copy link
Contributor Author

Merging now since we've had a few opinions and discussed it at the meeting.

https://lab.civicrm.org/documentation/docs/blob/master/meetings/notes/2018-04-03.md#style-guide

@seancolsen seancolsen merged commit 6bf533b into civicrm:master Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants