Updates ignoreSemantics migration guide#9124
Conversation
|
The main thing that's missing here is prose explaining why this feature existed in the first place, and why it is being removed. |
|
(e.g. the prose in flutter/flutter#131287) |
updated in the context section |
Co-authored-by: Shams Zakhour (ignore Sfshaza) <[email protected]>
| } | ||
| ``` | ||
|
|
||
| ## Timeline |
There was a problem hiding this comment.
Does you have a release defined for the timeline?
There was a problem hiding this comment.
this will be in the next release
There was a problem hiding this comment.
Can you provide info on when it was added to beta?
There was a problem hiding this comment.
Hi @sfshaza2, I think there were some confusions with ignoreSemantics and IgnorePointer in your previous comments. e.g. ignoreSemantics is not a widget, but a parameter in IgnorePointer.
I tried to follow the same structure you propose, but rephrase the sentences. Let me know if this looks good to you.
sfshaza2
left a comment
sfshaza2
left a comment
There was a problem hiding this comment.
Looks pretty good! I just found a couple of typos. And can you add this sort of info to timeline:
Landed in version: 3.11.0-0.0.pre<br>
In stable release: TBD
| } | ||
| ``` | ||
|
|
||
| ## Timeline |
There was a problem hiding this comment.
Can you provide info on when it was added to beta?
Co-authored-by: Shams Zakhour (ignore Sfshaza) <[email protected]>
it is already there in the file, or am i missing something? |
migration guide update flutter/website#9124 fixes the concerns in #120619
3 participants
relevant flutter/flutter#131287
Presubmit checklist