Skip to content

(Draft) banner#13396

Open
johnpryan wants to merge 1 commit into
mainfrom
banner-for-release
Open

(Draft) banner#13396
johnpryan wants to merge 1 commit into
mainfrom
banner-for-release

Conversation

@johnpryan
Copy link
Copy Markdown
Contributor

No description provided.

@johnpryan johnpryan requested review from a team and sfshaza2 as code owners May 18, 2026 17:52
Copy link
Copy Markdown
Member

@parlough parlough left a comment

Choose a reason for hiding this comment

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

Thanks John!

Requesting changes so this doesn't land on accident.

Copy link
Copy Markdown
Contributor

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

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 updates the site banner in site.yml to announce new Flutter features, replacing the previous Google I/O registration link. The review feedback suggests correcting a grammatical inconsistency in the banner text and optimizing the link by using a site-relative path and removing the "target="_blank"" attribute for better internal navigation.

Comment thread sites/docs/src/data/site.yml
Comment thread sites/docs/src/data/site.yml
@flutter-website-bot
Copy link
Copy Markdown
Collaborator

Visit the preview URL for this PR (updated for commit c9e6336):

https://flutter-docs-prod--pr13396-banner-for-release-8oi8s24i.web.app

Copy link
Copy Markdown
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm, so land when ready!

@parlough parlough self-assigned this May 18, 2026
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