Skip to content

Set canonical URL for every site page#13590

Merged
sfshaza2 merged 2 commits into
mainfrom
misc/set-canonical-url
Jul 18, 2026
Merged

Set canonical URL for every site page#13590
sfshaza2 merged 2 commits into
mainfrom
misc/set-canonical-url

Conversation

@parlough

@parlough parlough commented Jul 17, 2026

Copy link
Copy Markdown
Member

To comply with https://a14y.dev/scorecards/0.2.0/checks/html.canonical-link/

A page can specify a canonical in its YAML frontmatter to override the output canonical link.

@parlough
parlough requested review from a team, abdallahshaban557 and sfshaza2 as code owners July 17, 2026 23:08

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request ensures that a canonical URL is always resolved and included as a `<link rel=

Comment thread packages/site_shared/lib/src/layouts/dash_layout.dart
@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 1e5cbb2):

https://flutter-docs-prod--docs-pr13590-misc-set-canonical-url-6qy0bgkr.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit 1e5cbb2):

https://flutter-dev-230821--www-pr13590-misc-set-canonical-url-usb3ovgy.web.app

@abdallahshaban557 abdallahshaban557 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@sfshaza2 sfshaza2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@sfshaza2
sfshaza2 merged commit d280374 into main Jul 18, 2026
15 checks passed
@sfshaza2
sfshaza2 deleted the misc/set-canonical-url branch July 18, 2026 17:13
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