Skip to content

use meta.title as default <title> #454

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

eric-burel
Copy link
Contributor

Hey folks, long time no see!

After reviewing my sites SEO, I've noticed that the titles were not properly configured. Because TutorialKit also uses the title for navigation, it injects the chapter and part title too.

To sum it up the proposal

  • if meta.title is set, use it in the <title> tag: it will be picked up by search engines. But also visible in the browser window, so we lose the display of the part and chapter.
  • if not set, the <title> tag is just the navigational title part / chapter /lesson

Related but not included in this PR: I suspect trailingSlash: "never" is a reasonable default in the config, otherwise search engines might pick pages as duplicate. Might be worth an investigation.

Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

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.

1 participant