search

WordPress / wporg-main-2022

A block-based child theme for WordPress.org, plus local environment
62 stars 22 forks source link

SEO: Change separator to pipe. #384

Open StevenDufresne opened 5 months ago

StevenDufresne commented 5 months ago

Most of our subsites use a "|" as a separator. This PR updates the main theme to use the "|" separator as well.

This has come up as a result of working on https://github.com/WordPress/wporg-main-2022/pull/375.

Example URL Title
/patterns Block Pattern Directory | WordPress.org
/showcase Star-studded sites built with WordPress | WordPress.org
/plugins WordPress Plugins | WordPress.org

Currently, the main theme doesn't enforce that pattern and we get the default "-" separator.

StevenDufresne commented 5 months ago

Looks good.

If this is the expected format, maybe it's something we should move to site-branding.php mu-plugin?

Yeah, that's a good call. I move this change over there and close this PR.