Skip to content
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

Picture.toImage breaks backwards compatibility #26655

Closed
lukef opened this issue Jan 16, 2019 · 6 comments
Closed

Picture.toImage breaks backwards compatibility #26655

lukef opened this issue Jan 16, 2019 · 6 comments
Labels
c: API break Backwards-incompatible API changes engine flutter/engine repository. See also e: labels.

Comments

@lukef
Copy link
Contributor

lukef commented Jan 16, 2019

On master, Picture.toImage in dart:ui now returns a Future (so is async) where it used to be sync. This creates an issue with libraries that use this call (like https://github.com/lukef/qr.flutter).

I'm going to assume that there are going to be some assurances around backwards compatibility moving forward? If not, we will need a way to isolate platform version specific code so that developers (like me) don't have to maintain a bunch of different library versions whenever the underlying platform changes.

For now, I can bump the version and direct developers to use a lower version for 1.0 and a higher version for master (and beyond) however, should we trust that decisions like this one that are made (and pushed) to master are going to be in constant flux? i.e: tell developers to not use master?

Also, there is no mechanism to peg the library to specific Flutter versions (right?). So it's still highly possible that developers will use the wrong version regardless.

@GaryQian
Copy link
Contributor

cc @jason-simmons

@GaryQian GaryQian added engine flutter/engine repository. See also e: labels. c: API break Backwards-incompatible API changes labels Jan 16, 2019
@zoechi
Copy link
Contributor

zoechi commented Jan 16, 2019

As far as I know this was to fix #23621 by #7442 to make Picture.toImage work.

@lukef
Copy link
Contributor Author

lukef commented Jan 16, 2019

I have to assume that this fix wont be rolled back at this point?

@jason-simmons
Copy link
Member

This is an intentional change to the API. Fixing #23621 required making this method asynchronous

@zoechi
Copy link
Contributor

zoechi commented Jan 17, 2019

decisions like this one that are made (and pushed) to master are going to be in constant flux?

From what I have seen the Flutter team avoids any breaking change, but in this case it was necessary to make a broken feature work.

I don't think there is anything actionable left and therefore closing.

@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
c: API break Backwards-incompatible API changes engine flutter/engine repository. See also e: labels.
Projects
None yet
Development

No branches or pull requests

4 participants