Add header migration guide#13457
Conversation
![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 adds documentation for a breaking change regarding the update of semantics header and headingLevel behavior on iOS and Android, including a new markdown file and an update to the breaking changes index. Feedback on the changes points out extra blank lines in the index file and a conflicting ID reference in the references section where both an issue and a PR are listed with the same ID (175416).
| Relevant PRs: | ||
|
|
||
| * [PR 186916][] | ||
| * [PR 175416][] |
There was a problem hiding this comment.
GitHub issues and pull requests share the same ID sequence within a repository, meaning Issue 175416 and PR 175416 cannot both exist. One of these references is incorrect and should be updated to the correct ID (also update the corresponding link definition on line 119).
|
Staged preview of the updated docs.flutter.dev site (updated for commit ac0c429): |
parlough
left a comment
parlough
left a comment
There was a problem hiding this comment.
Thanks! Looks good to me.
Feel free to land if it's ready. You can ignore the www-preview-bot failure.
3 participants
migration guide for flutter/flutter#186916
Presubmit checklist
of 80 characters or fewer.