Change the banner content to promote Q2 survey#13441
Conversation
Updated bannerHtml to promote Q2 survey.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the site banner in sites/docs/src/data/site.yml to promote the Q2 survey instead of the Flutter 3.44 release. The review feedback suggests improving the link by removing the first-person pronoun 'our' to comply with style guidelines and adding rel="noopener" to the anchor tag for security.
| Watch <a href="https://www.youtube.com/watch?v=I1uIbGh1dGE&t=1s" target="_blank">What's new in Flutter</a> and | ||
| read the <a href="https://blog.flutter.dev/whats-new-in-flutter-3-44-b0cc1ad3c527" target="_blank">blog post</a>. | ||
| Help improve Flutter! | ||
| <a href="https://google.qualtrics.com/jfe/form/SV_3drKjSfjNeLZfq6?Source=Website" target="_blank">Take our Q2 survey</a>. |
There was a problem hiding this comment.
To adhere to the Google Developer Documentation Style Guidelines mentioned in the PR checklist, avoid using the first-person pronoun 'our' by changing it to 'the'. Additionally, for security, add rel="noopener" to the target="_blank" link to prevent potential reverse tabnabbing.
<a href="https://google.qualtrics.com/jfe/form/SV_3drKjSfjNeLZfq6?Source=Website" target="_blank" rel="noopener">Take the Q2 survey</a>.
|
Staged preview of the updated docs.flutter.dev site (updated for commit e931e56): https://flutter-docs-prod--docs-pr13441-zanderso-patch-1-t819w29n.web.app |
|
Oops. This wasn't supposed to land until Monday =) I will click the revert button. |
3 participants
Description of what this PR is changing or adding, and why:
The survey is planned to run from 6/1 to 6/12. This PR should not land before 6/1.
Issues fixed by this PR (if any):
PRs or commits this PR depends on (if any):
Presubmit checklist
of 80 characters or fewer.