Skip to content
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

(DOCSP-29016): Added SEO guidelines. #144

Merged
merged 5 commits into from
May 1, 2023

Conversation

corryroot
Copy link
Contributor

@corryroot corryroot commented Apr 11, 2023

Pull Request Process

Please read the
Style Guide Review Process
wiki page.

Contributors should take the following actions:

  • Request a PR review in #docs-style-guide Slack channel.
  • Add the Style Guide team reps as reviewers.
  • If any reviewer requests changes, address them and request another review.
  • When you receive PR review approvals from all the Style Guide team reps,
    they will merge the PR and notify you of the merge in the Slack channel.

Pull Request Description

I created this PR in relation to #118. I incorporated the guidance that Cloud Docs received from the SEO team.

JIRA

STAGE

BUILD

@corryroot corryroot added the copy review Review for language, format, and structure label Apr 11, 2023
@corryroot corryroot force-pushed the DOCSP-29016 branch 2 times, most recently from 91d64e8 to c0ffca5 Compare April 12, 2023 13:22
Copy link
Contributor

@lindseymoore lindseymoore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@ccho-mongodb ccho-mongodb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks mostly good, found some small issues and descriptions that could use more detail.

source/style-guide.txt Outdated Show resolved Hide resolved
source/style-guide/markup/directives/metadata.txt Outdated Show resolved Hide resolved
source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved
source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved
source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved
Comment on lines +48 to +51
You can add the actual terms that users enter into search engines
(keywords) to your page to improve its SEO. Add keywords according to
the following best practices:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion:

I think it might be better to suggest that keywords may not help SEO nor be used to match up with search terms, at least with Google. See
https://developers.google.com/search/docs/crawling-indexing/special-tags#unsupported
for more info.

Copy link
Contributor Author

@corryroot corryroot Apr 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, there are two parts to these recommendations. We should use the language that users search for in our content. But, search engines might not account for the keyword tags that we use.

source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved

- Use a maximum of 155 characters.

- Include target keywords and a call to action.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question:
I thought Call to Action (CTA) is usually a UI element such as a button that contains text that prompts a user to do do something such as click. I don't think that's something you can include in a text tag. If it's something else, could that be explained here?

Copy link
Contributor Author

@corryroot corryroot Apr 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We want to call the user to action with the description that appears in the search result.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a blocker for me, but maybe including an example could be helpful. Does this mean including text like "Click here to learn more" in the meta description?

source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved
source/style-guide/seo-guidelines.txt Outdated Show resolved Hide resolved
Copy link
Contributor

@ccho-mongodb ccho-mongodb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Added one suggestion to add an example.

Copy link
Collaborator

@jeff-allen-mongo jeff-allen-mongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@corryroot thanks for this update. As someone who knows very little about SEO, this is super helpful and I learned a lot reading it. I left a couple comments and suggestions for your consideration.

Jeff

========================
Page Metadata Directives
========================

meta
----

Generates HTML ``<meta>`` tags.
Use ``.. meta::`` to add |html| meta tags to a page. Adding meta tags
aids in searches both internally and via the Internet. You can add both
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[issue]

Our style guide suggests avoiding "via". Maybe consider "through the Internet" instead?

Style guide link: https://www.mongodb.com/docs/meta/style-guide/terminology/alphabetical-terms/#v

Titles
------

Title pages and subsections according to the following SEO best
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[suggestion, wording]

It wasn't immediately clear to me that we're using "title" as a verb in this sentence. On initial reading I thought "title page" was a noun. Perhaps we can tweak this to something like:

"Use the following SEO best practices for page and subsection titles: "

Comment on lines +53 to +54
- Take the most concise form of the information that the page conveys
and make that the target keyword.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[suggestion, non-blocking]

Would it be possible to include an example of this practice? For instance, I would find it helpful if we provided a link to some docs page and highlighted what the target keyword would be for that page.

Comment on lines 41 to 42
- If it makes sense, include keywords (the actual terms that users
enter into search engines).
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[question]

The phrase "if it makes sense" makes me wonder in which situations this practice doesn't make sense. Is that something we can possibly clarify a bit here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it's a question about whether the keywords for the page really apply to the image. I can think of examples where they might not. I'll change the wording to "If they apply to the image..."

Copy link
Collaborator

@jeff-allen-mongo jeff-allen-mongo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@corryroot corryroot removed the copy review Review for language, format, and structure label Apr 28, 2023
@corryroot
Copy link
Contributor Author

Elizabeth Baggan approved this PR on Slack.

@corryroot corryroot merged commit 8e55d25 into mongodb:master May 1, 2023
@corryroot corryroot deleted the DOCSP-29016 branch May 1, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants