Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix compilation on Flutter 3.13.0 stable #915

Merged
merged 4 commits into from
Aug 17, 2023
Merged

Fix compilation on Flutter 3.13.0 stable #915

merged 4 commits into from
Aug 17, 2023

Conversation

h3x4d3c1m4l
Copy link
Contributor

This PR (rudimentarily) fixes compilation on Flutter 3.13.0 stable. I tested running the example on Windows, it runs and I haven't found anything broken yet.

However, as the breaking changed APIs were not documented (here or here) and knowing my knowledge of both Fluent UI's and Flutter's internals is limited, these changes might not be the correct...

Pre-launch Checklist

  • I have updated CHANGELOG.md with my changes
  • I have run "dart format ." on the project
  • I have added/updated relevant documentation

@h3x4d3c1m4l
Copy link
Contributor Author

Ah, analysis fails due to ignoringSemantics being deprecated. However I can't seem to find the equivalent of the current implementation...

I don't think ExcludeSemantics widget will do as it will cause the opposite effect, right?

Copy link
Owner

@bdlukaa bdlukaa left a comment

Choose a reason for hiding this comment

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

The mentioned line with ignoringSemantics can be removed. The semantics should be included automatically after its deprecation!

@bdlukaa bdlukaa linked an issue Aug 17, 2023 that may be closed by this pull request
@bdlukaa bdlukaa merged commit 1375c28 into bdlukaa:master Aug 17, 2023
@h3x4d3c1m4l h3x4d3c1m4l deleted the compile-fixes-for-flutter-3-13 branch August 17, 2023 17:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Fluent UI 4.6.2 breaks under Flutter 3.12.0-8.0.pre.21
3 participants