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
more on Flutter web indexability #10550
Conversation
Visit the preview URL for this PR (updated for commit b204f69): https://flutter-docs-prod--pr10550-more-on-seo-g0uq1szw.web.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing wrong with the occasion "we", but I suggested a change to not add as many.
Acceptance is your call.
Having said that, as we mention in our [roadmap][], we plan to investigate | ||
search engine indexability of Flutter Web. To that end we've built a small | ||
website containing [Hawaii-themed space stories][space_hawaii], which we | ||
would like search engines to pick up and index. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: Would recommend tightening somewhat as the quantity of "we"s is a bit much.
Having said that, as we mention in our [roadmap][], we plan to investigate | |
search engine indexability of Flutter Web. To that end we've built a small | |
website containing [Hawaii-themed space stories][space_hawaii], which we | |
would like search engines to pick up and index. | |
That said, as mentioned in the [roadmap][], the Flutter team plans to investigate | |
search engine indexability of Flutter Web. | |
To that end, we built a small website containing [Hawaii-themed space stories][space_hawaii]. | |
We intend for search engines to find and index this site. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Further feedback on Tony's feedback:
- It's "Flutter web" NOT "Flutter Web"
- "To that end, we built a small website containing [Hawaii-themed space stories][space_hawaii].
We intend for search engines to find and index this site." =>
To that end, we built a small website containing [Hawaii-themed space stories][space_hawaii],
hoping that search engines find and index this site."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done. Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor tweaks, Yegor, but I LOVE this clever idea! I'll go ahead and approve and you can land this whenever you are ready.
Having said that, as we mention in our [roadmap][], we plan to investigate | ||
search engine indexability of Flutter Web. To that end we've built a small | ||
website containing [Hawaii-themed space stories][space_hawaii], which we | ||
would like search engines to pick up and index. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Further feedback on Tony's feedback:
- It's "Flutter web" NOT "Flutter Web"
- "To that end, we built a small website containing [Hawaii-themed space stories][space_hawaii].
We intend for search engines to find and index this site." =>
To that end, we built a small website containing [Hawaii-themed space stories][space_hawaii],
hoping that search engines find and index this site."
Let's hold off on merging this until we can find a solution for b/339684234 |
Can we get this landed, @yjbanov? |
I think we're okay to land now, @sfshaza2 ! |
Expand on what we're doing about indexability.
One unusual aspect of this change is linking to the test site and spelling out "Hawaii-themed space stories". This is intentional, and hopefully, not too disruptive to the message. In order to prove that SEO works we need an already indexed website to link to the test site so search engines find it and attempt to index it. I thought sneaking one such link into a section of our web FAQ that already talks about SEO would be a safe place for it.
One place I violate the style guide is I use "we" in the text. However, it seems to be consistent with how the word is used elsewhere on the site. I'm open to suggestions for how to remove it, if it's important.
Presubmit checklist