Remove styleguide page from acceptableads.com?

[juliandoucette]

Background

The ​styleguide page is unlisted on acceptableads.com and we have not been maintaining it.

What to change

Remove this page and it's includes.

[juliandoucette]

I'm not sure if/how we could/should integrate such a page into website-defaults and/or individual websites.

What do you think Ire?

[ire]

I don't necessarily think it should be removed because we have not been maintaining it. But maybe it should be changed.

I think that perhaps the reason it has not been maintained much is because it seems to be a relatively "full-featured" style guide. It does not only include, like most style guides, basic things such as headings, colours, and some components like buttons. Instead, it seems to be more a guide that non-technical people can follow to build pages themselves. This may be more useful but is also more a hassle to maintain.

It may be simpler to have a more minimal style guide as a part of website-defaults that will include those basic elements that are on every site. For example, I can imagine having files like the headings.md one as part of website-defaults that could be included into a site's style guide. And if necessary, specific sites can have additional elements that are more specific to the site.

[juliandoucette]

Replying to iaderinokun:

It may be simpler to have a more minimal style guide as a part of website-defaults that will include those basic elements that are on every site. For example, I can imagine having files like the headings.md one as part of website-defaults that could be included into a site's style guide. And if necessary, specific sites can have additional elements that are more specific to the site.

Sounds good to me.

How do you feel about implementing such a website-defaults guide while you are implementing help center?

If you don't mind, we'll:

1. Leave this be on acceptableads.com
2. Implement a basic style guide in website-defaults
3. Extend said basic guide in help center
4. Extend said basic guide in acceptableads.com (if we ever get around to it)

And you can create the tickets for 2 and 3.

Note: We can't include directly from website-defaults yet via CMS. Therefore, you could include the one that will go into website-defaults in help center until this issue is resolved.

[ire]

Replying to juliandoucette:

Sounds good to me.

How do you feel about implementing such a website-defaults guide while you are implementing help center?

If you don't mind, we'll:

1. Leave this be on acceptableads.com
2. Implement a basic style guide in website-defaults
3. Extend said basic guide in help center
4. Extend said basic guide in acceptableads.com (if we ever get around to it)

And you can create the tickets for 2 and 3.

Note: We can't include directly from website-defaults yet via CMS. Therefore, you could include the one that will go into website-defaults in help center until this issue is resolved.

Sure sounds good.

[ire]

Since we implemented V1 of a styleguide page in #5413, I think we can remove the aa custom styleguide pages, and improve the one in wd

[juliandoucette]

I agree. Thank you for your feedback. I will wait until the new styleguide is fully implemented before removing this one though. It may be useful yet. Let's tackle this / styleguide integration separately on Gitlab.